Add tidy-alphabetical-start/end around feature lists and, in some cases, allow/warn/deny lists

Author: clarfonthey

compiler/rustc_arena/src/lib.rs

@@ -11,20 +11,23 @@
     html_root_url = "https://doc.rust-lang.org/nightly/nightly-rustc/",
     test(no_crate_inject, attr(deny(warnings)))
 )]
+// tidy-alphabetical-start
+#![deny(rustc::diagnostic_outside_of_impl)]
+#![deny(rustc::untranslatable_diagnostic)]
+#![deny(unsafe_op_in_unsafe_fn)]
+#![feature(decl_macro)]
 #![feature(dropck_eyepatch)]
-#![feature(new_uninit)]
 #![feature(maybe_uninit_slice)]
 #![feature(min_specialization)]
-#![feature(decl_macro)]
+#![feature(new_uninit)]
 #![feature(pointer_byte_offsets)]
 #![feature(rustc_attrs)]
-#![cfg_attr(test, feature(test))]
 #![feature(strict_provenance)]
-#![deny(unsafe_op_in_unsafe_fn)]
-#![deny(rustc::untranslatable_diagnostic)]
-#![deny(rustc::diagnostic_outside_of_impl)]
+// tidy-alphabetical-end
 #![cfg_attr(not(bootstrap), allow(internal_features))]
-#![allow(clippy::mut_from_ref)] // Arena allocators are one of the places where this pattern is fine.
+#![cfg_attr(test, feature(test))]
+// Arena allocators are one of the places where this pattern is fine.
+#![allow(clippy::mut_from_ref)]
 
 use smallvec::SmallVec;
 

compiler/rustc_ast/src/lib.rs

@@ -8,6 +8,9 @@
     html_root_url = "https://doc.rust-lang.org/nightly/nightly-rustc/",
     test(attr(deny(warnings)))
 )]
+// tidy-alphabetical-start
+#![deny(rustc::diagnostic_outside_of_impl)]
+#![deny(rustc::untranslatable_diagnostic)]
 #![feature(associated_type_bounds)]
 #![feature(box_patterns)]
 #![feature(const_trait_impl)]
@@ -16,9 +19,8 @@
 #![feature(min_specialization)]
 #![feature(negative_impls)]
 #![feature(stmt_expr_attributes)]
+// tidy-alphabetical-end
 #![recursion_limit = "256"]
-#![deny(rustc::untranslatable_diagnostic)]
-#![deny(rustc::diagnostic_outside_of_impl)]
 
 #[macro_use]
 extern crate rustc_macros;

compiler/rustc_ast_lowering/src/lib.rs

@@ -30,12 +30,14 @@
 //! get confused if the spans from leaf AST nodes occur in multiple places
 //! in the HIR, especially for multiple identifiers.
 
+// tidy-alphabetical-start
+#![deny(rustc::diagnostic_outside_of_impl)]
+#![deny(rustc::untranslatable_diagnostic)]
 #![feature(box_patterns)]
 #![feature(let_chains)]
 #![feature(never_type)]
+// tidy-alphabetical-end
 #![recursion_limit = "256"]
-#![deny(rustc::untranslatable_diagnostic)]
-#![deny(rustc::diagnostic_outside_of_impl)]
 
 #[macro_use]
 extern crate tracing;

compiler/rustc_ast_passes/src/lib.rs

@@ -4,10 +4,12 @@
 //!
 //! The crate also contains other misc AST visitors, e.g. `node_count` and `show_span`.
 
+// tidy-alphabetical-start
 #![feature(box_patterns)]
 #![feature(if_let_guard)]
 #![feature(iter_is_partitioned)]
 #![feature(let_chains)]
+// tidy-alphabetical-end
 #![recursion_limit = "256"]
 #![deny(rustc::untranslatable_diagnostic)]
 #![deny(rustc::diagnostic_outside_of_impl)]

compiler/rustc_ast_pretty/src/lib.rs

@@ -1,8 +1,10 @@
-#![deny(rustc::untranslatable_diagnostic)]
+// tidy-alphabetical-start
 #![deny(rustc::diagnostic_outside_of_impl)]
+#![deny(rustc::untranslatable_diagnostic)]
 #![feature(associated_type_bounds)]
 #![feature(box_patterns)]
 #![feature(with_negative_coherence)]
+// tidy-alphabetical-end
 #![recursion_limit = "256"]
 
 mod helpers;

compiler/rustc_attr/src/lib.rs

@@ -4,9 +4,11 @@
 //! The goal is to move the definition of `MetaItem` and things that don't need to be in `syntax`
 //! to this crate.
 
-#![feature(let_chains)]
-#![deny(rustc::untranslatable_diagnostic)]
+// tidy-alphabetical-start
 #![deny(rustc::diagnostic_outside_of_impl)]
+#![deny(rustc::untranslatable_diagnostic)]
+#![feature(let_chains)]
+// tidy-alphabetical-end
 
 #[macro_use]
 extern crate rustc_macros;

compiler/rustc_borrowck/src/lib.rs

@@ -1,15 +1,17 @@
 //! This query borrow-checks the MIR to (further) ensure it is not broken.
 
+// tidy-alphabetical-start
 #![feature(associated_type_bounds)]
 #![feature(box_patterns)]
+#![feature(lazy_cell)]
 #![feature(let_chains)]
 #![feature(min_specialization)]
 #![feature(never_type)]
-#![feature(lazy_cell)]
 #![feature(rustc_attrs)]
 #![feature(stmt_expr_attributes)]
 #![feature(trusted_step)]
 #![feature(try_blocks)]
+// tidy-alphabetical-end
 #![recursion_limit = "256"]
 #![cfg_attr(not(bootstrap), allow(internal_features))]
 

compiler/rustc_builtin_macros/src/lib.rs

@@ -2,6 +2,7 @@
 //! injecting code into the crate before it is lowered to HIR.
 
 #![doc(html_root_url = "https://doc.rust-lang.org/nightly/nightly-rustc/")]
+// tidy-alphabetical-start
 #![feature(array_windows)]
 #![feature(box_patterns)]
 #![feature(decl_macro)]
@@ -11,6 +12,7 @@
 #![feature(lint_reasons)]
 #![feature(proc_macro_internals)]
 #![feature(proc_macro_quote)]
+// tidy-alphabetical-end
 #![recursion_limit = "256"]
 
 extern crate proc_macro;

compiler/rustc_codegen_cranelift/src/lib.rs

@@ -1,8 +1,10 @@
-#![feature(rustc_private)]
 // Note: please avoid adding other feature gates where possible
+#![feature(rustc_private)]
+// tidy-alphabetical-start
 #![warn(rust_2018_idioms)]
-#![warn(unused_lifetimes)]
 #![warn(unreachable_pub)]
+#![warn(unused_lifetimes)]
+// tidy-alphabetical-end
 
 extern crate jobserver;
 #[macro_use]

compiler/rustc_codegen_gcc/src/lib.rs

@@ -6,20 +6,22 @@
  * TODO(antoyo): remove the patches.
  */
 
-#![feature(
-    rustc_private,
-    decl_macro,
-    associated_type_bounds,
-    never_type,
-    trusted_len,
-    hash_raw_entry
-)]
+// tidy-alphabetical-start
 #![allow(broken_intra_doc_links)]
-#![recursion_limit="256"]
+#![deny(rustc::diagnostic_outside_of_impl)]
+#![deny(rustc::untranslatable_diagnostic)]
 #![warn(rust_2018_idioms)]
 #![warn(unused_lifetimes)]
-#![deny(rustc::untranslatable_diagnostic)]
-#![deny(rustc::diagnostic_outside_of_impl)]
+// tidy-alphabetical-end
+// tidy-alphabetical-start
+#![feature(associated_type_bounds)]
+#![feature(decl_macro)]
+#![feature(hash_raw_entry)]
+#![feature(never_type)]
+#![feature(rustc_private)]
+#![feature(trusted_len)]
+// tidy-alphabetical-end
+#![recursion_limit="256"]
 
 extern crate rustc_apfloat;
 extern crate rustc_ast;