Remove #[no_link] attribute

Author: bjorn3

compiler/rustc_attr_parsing/src/attributes/mod.rs

@@ -58,7 +58,6 @@ pub(crate) mod macro_attrs;
 pub(crate) mod must_not_suspend;
 pub(crate) mod must_use;
 pub(crate) mod no_implicit_prelude;
-pub(crate) mod no_link;
 pub(crate) mod non_exhaustive;
 pub(crate) mod path;
 pub(crate) mod pin_v2;

compiler/rustc_attr_parsing/src/attributes/no_link.rs

@@ -46,7 +46,6 @@ use crate::attributes::macro_attrs::*;
 use crate::attributes::must_not_suspend::*;
 use crate::attributes::must_use::*;
 use crate::attributes::no_implicit_prelude::*;
-use crate::attributes::no_link::*;
 use crate::attributes::non_exhaustive::*;
 use crate::attributes::path::PathParser as PathAttributeParser;
 use crate::attributes::pin_v2::*;
@@ -252,7 +251,6 @@ attribute_parsers!(
         Single<WithoutArgs<NoBuiltinsParser>>,
         Single<WithoutArgs<NoCoreParser>>,
         Single<WithoutArgs<NoImplicitPreludeParser>>,
-        Single<WithoutArgs<NoLinkParser>>,
         Single<WithoutArgs<NoMainParser>>,
         Single<WithoutArgs<NoMangleParser>>,
         Single<WithoutArgs<NoStdParser>>,

compiler/rustc_attr_parsing/src/context.rs

@@ -114,7 +114,6 @@ pub static BUILTIN_ATTRIBUTES: &[Symbol] = &[
     // ABI, linking, symbols, and FFI
     sym::link,
     sym::link_name,
-    sym::no_link,
     sym::repr,
     // FIXME(#82232, #143834): temporarily renamed to mitigate `#[align]` nameres ambiguity
     sym::rustc_align,

compiler/rustc_feature/src/builtin_attrs.rs

@@ -1146,9 +1146,6 @@ pub enum AttributeKind {
     /// Represents `#[no_implicit_prelude]`
     NoImplicitPrelude,
 
-    /// Represents `#[no_link]`
-    NoLink,
-
     /// Represents `#[no_main]`
     NoMain,
 

compiler/rustc_hir/src/attrs/data_structures.rs

@@ -71,7 +71,6 @@ impl AttributeKind {
             NoBuiltins => Yes,
             NoCore => No,
             NoImplicitPrelude => No,
-            NoLink => No,
             NoMain => No,
             NoMangle(..) => Yes, // Needed for rustdoc
             NoStd => No,

compiler/rustc_hir/src/attrs/encode_cross_crate.rs

@@ -1333,11 +1333,7 @@ impl CStore {
                     }
                     None => ident.name,
                 };
-                let dep_kind = if attr::contains_name(&item.attrs, sym::no_link) {
-                    CrateDepKind::MacrosOnly
-                } else {
-                    CrateDepKind::Unconditional
-                };
+                let dep_kind = CrateDepKind::Unconditional;
 
                 let cnum =
                     self.resolve_crate(tcx, name, item.span, dep_kind, CrateOrigin::Extern)?;

compiler/rustc_metadata/src/creader.rs

@@ -293,7 +293,6 @@ impl<'tcx> CheckAttrVisitor<'tcx> {
             AttributeKind::NoBuiltins => (),
             AttributeKind::NoCore { .. } => (),
             AttributeKind::NoImplicitPrelude => (),
-            AttributeKind::NoLink => (),
             AttributeKind::NoMain => (),
             AttributeKind::NoMangle(..) => (),
             AttributeKind::NoStd { .. } => (),

compiler/rustc_passes/src/check_attr.rs

@@ -157,8 +157,6 @@ impl Stop {
             sym::must_use |
             // Should be applied to a foreign function or static
             sym::link_name | sym::link_ordinal | sym::link_section |
-            // Should be applied to an `extern crate` item
-            sym::no_link |
             // Should be applied to a free function, impl method or static
             sym::export_name | sym::no_mangle |
             // Should be applied to a `static` variable

src/tools/clippy/clippy_lints/src/empty_line_after.rs

@@ -186,7 +186,6 @@ pub const INERT_ATTRIBUTES: &[BuiltinAttribute] = &[
         DuplicatesOk,
     ),
     ungated!(link_name, Normal, template!(NameValueStr: "name"), FutureWarnPreceding),
-    ungated!(no_link, Normal, template!(Word), WarnFollowing),
     ungated!(repr, Normal, template!(List: "C"), DuplicatesOk, @only_local: true),
     ungated!(export_name, Normal, template!(NameValueStr: "name"), FutureWarnPreceding),
     ungated!(link_section, Normal, template!(NameValueStr: "name"), FutureWarnPreceding),