Add lint rule for #[deprecated] on use

Author: kailan

compiler/rustc_passes/src/stability.rs

@@ -366,6 +366,9 @@ impl<'a, 'tcx> Visitor<'tcx> for Annotator<'a, 'tcx> {
                 kind = AnnotationKind::DeprecationProhibited;
                 const_stab_inherit = InheritConstStability::Yes;
             }
+            hir::ItemKind::Use(_, _) => {
+                kind = AnnotationKind::DeprecationProhibited;
+            }
             hir::ItemKind::Struct(ref sd, _) => {
                 if let Some(ctor_def_id) = sd.ctor_def_id() {
                     self.annotate(

library/core/src/alloc/mod.rs

@@ -10,11 +10,6 @@ pub use self::global::GlobalAlloc;
 #[stable(feature = "alloc_layout", since = "1.28.0")]
 pub use self::layout::Layout;
 #[stable(feature = "alloc_layout", since = "1.28.0")]
-#[deprecated(
-    since = "1.52.0",
-    note = "Name does not follow std convention, use LayoutError",
-    suggestion = "LayoutError"
-)]
 #[allow(deprecated, deprecated_in_future)]
 pub use self::layout::LayoutErr;
 #[stable(feature = "alloc_layout_error", since = "1.50.0")]

library/std/src/collections/mod.rs

@@ -433,8 +433,7 @@ pub use self::hash_map::HashMap;
 #[doc(inline)]
 pub use self::hash_set::HashSet;
 #[stable(feature = "rust1", since = "1.0.0")]
-// FIXME(#82080) The deprecation here is only theoretical, and does not actually produce a warning.
-#[deprecated(note = "moved to `std::ops::Bound`", since = "1.26.0")]
+// FIXME(#82080) This has moved but #[deprecated] on `use` is unsupported.
 #[doc(hidden)]
 pub use crate::ops::Bound;
 

tests/ui/deprecation/deprecated_use.fixed

@@ -0,0 +1,10 @@
+//@ run-rustfix
+mod a_module {
+  pub struct ActiveType;
+}
+
+//~^ ERROR this `#[deprecated]` annotation has no effect
+pub use a_module::ActiveType;
+
+fn main() {
+}

tests/ui/deprecation/deprecated_use.rs

@@ -0,0 +1,11 @@
+//@ run-rustfix
+mod a_module {
+  pub struct ActiveType;
+}
+
+#[deprecated]
+//~^ ERROR this `#[deprecated]` annotation has no effect
+pub use a_module::ActiveType;
+
+fn main() {
+}

tests/ui/deprecation/deprecated_use.stderr

@@ -0,0 +1,10 @@
+error: this `#[deprecated]` annotation has no effect
+  --> $DIR/deprecated_use.rs:6:1
+   |
+LL | #[deprecated]
+   | ^^^^^^^^^^^^^ help: remove the unnecessary deprecation attribute
+   |
+   = note: `#[deny(useless_deprecated)]` on by default
+
+error: aborting due to 1 previous error
+

tests/ui/imports/unused-import-issue-87973.fixed

@@ -1,5 +1,6 @@
 //@ run-rustfix
 #![deny(unused_imports)]
+#![allow(useless_deprecated)]
 
 // Check that attributes get removed too. See #87973.
 //~^ ERROR unused import

tests/ui/imports/unused-import-issue-87973.rs

@@ -1,5 +1,6 @@
 //@ run-rustfix
 #![deny(unused_imports)]
+#![allow(useless_deprecated)]
 
 // Check that attributes get removed too. See #87973.
 #[deprecated]

tests/ui/imports/unused-import-issue-87973.stderr

@@ -1,5 +1,5 @@
 error: unused import: `std::fs`
-  --> $DIR/unused-import-issue-87973.rs:8:5
+  --> $DIR/unused-import-issue-87973.rs:9:5
    |
 LL | use std::fs;
    |     ^^^^^^^