Follow-up cleanups reusing the now-available LocalDefIds

oli-obk · oli-obk · commit 0a151cae895f · 2026-05-05T07:32:44.000+02:00
diff --git a/compiler/rustc_resolve/src/check_unused.rs b/compiler/rustc_resolve/src/check_unused.rs
@@ -431,8 +431,7 @@ impl Resolver<'_, '_> {
                         }
                     }
                 }
-                ImportKind::ExternCrate { id, .. } => {
-                    let def_id = self.local_def_id(id);
+                ImportKind::ExternCrate { id, def_id, .. } => {
                     if self.extern_crate_map.get(&def_id).is_none_or(|&cnum| {
                         !tcx.is_compiler_builtins(cnum)
                             && !tcx.is_panic_runtime(cnum)
diff --git a/compiler/rustc_resolve/src/imports.rs b/compiler/rustc_resolve/src/imports.rs
@@ -346,13 +346,16 @@ struct UnresolvedImportError {
 
 // Reexports of the form `pub use foo as bar;` where `foo` is `extern crate foo;`
 // are permitted for backward-compatibility under a deprecation lint.
-fn pub_use_of_private_extern_crate_hack(import: ImportSummary, decl: Decl<'_>) -> Option<NodeId> {
+fn pub_use_of_private_extern_crate_hack(
+    import: ImportSummary,
+    decl: Decl<'_>,
+) -> Option<LocalDefId> {
     match (import.is_single, decl.kind) {
         (true, DeclKind::Import { import: decl_import, .. })
-            if let ImportKind::ExternCrate { id, .. } = decl_import.kind
+            if let ImportKind::ExternCrate { def_id, .. } = decl_import.kind
                 && import.vis.is_public() =>
         {
-            Some(id)
+            Some(def_id)
         }
         _ => None,
     }
@@ -1240,7 +1243,7 @@ impl<'ra, 'tcx> Resolver<'ra, 'tcx> {
 
         let (ident, target, bindings, import_id) = match import.kind {
             ImportKind::Single { source, target, ref decls, id, .. } => (source, target, decls, id),
-            ImportKind::Glob { ref max_vis, id, def_id: _ } => {
+            ImportKind::Glob { ref max_vis, id, def_id } => {
                 if import.module_path.len() <= 1 {
                     // HACK(eddyb) `lint_if_path_starts_with_module` needs at least
                     // 2 segments, so the `resolve_path` above won't trigger it.
@@ -1267,7 +1270,6 @@ impl<'ra, 'tcx> Resolver<'ra, 'tcx> {
                 if let Some(max_vis) = max_vis.get()
                     && import.vis.greater_than(max_vis, self.tcx)
                 {
-                    let def_id = self.local_def_id(id);
                     self.lint_buffer.buffer_lint(
                         UNUSED_IMPORTS,
                         id,
@@ -1549,7 +1551,7 @@ impl<'ra, 'tcx> Resolver<'ra, 'tcx> {
             if let Some(extern_crate_id) =
                 pub_use_of_private_extern_crate_hack(import.summary(), binding)
             {
-                let extern_crate_sp = self.tcx.source_span(self.local_def_id(extern_crate_id));
+                let extern_crate_sp = self.tcx.source_span(extern_crate_id);
                 self.lint_buffer.buffer_lint(
                     PUB_USE_OF_PRIVATE_EXTERN_CRATE,
                     import_id,
@@ -1625,7 +1627,7 @@ impl<'ra, 'tcx> Resolver<'ra, 'tcx> {
 
     pub(crate) fn check_for_redundant_imports(&mut self, import: Import<'ra>) -> bool {
         // This function is only called for single imports.
-        let ImportKind::Single { source, target, ref decls, id, .. } = import.kind else {
+        let ImportKind::Single { source, target, ref decls, id, def_id, .. } = import.kind else {
             unreachable!()
         };
 
@@ -1644,7 +1646,7 @@ impl<'ra, 'tcx> Resolver<'ra, 'tcx> {
         // Skip if the import is public or was used through non scope-based resolution,
         // e.g. through a module-relative path.
         if self.import_use_map.get(&import) == Some(&Used::Other)
-            || self.effective_visibilities.is_exported(self.local_def_id(id))
+            || self.effective_visibilities.is_exported(def_id)
         {
             return false;
         }
diff --git a/compiler/rustc_resolve/src/lib.rs b/compiler/rustc_resolve/src/lib.rs
@@ -2282,8 +2282,7 @@ impl<'ra, 'tcx> Resolver<'ra, 'tcx> {
 
     #[inline]
     fn add_to_glob_map(&mut self, import: Import<'_>, name: Symbol) {
-        if let ImportKind::Glob { id, .. } = import.kind {
-            let def_id = self.local_def_id(id);
+        if let ImportKind::Glob { def_id, .. } = import.kind {
             self.glob_map.entry(def_id).or_default().insert(name);
         }
     }

Original file line number	Diff line number	Diff line change
`@@ -431,8 +431,7 @@ impl Resolver<'_, '_> {`
`431`	`431`	`}`
`432`	`432`	`}`
`433`	`433`	`}`
`434`		`- ImportKind::ExternCrate { id, .. } => {`
`435`		`- let def_id = self.local_def_id(id);`
	`434`	`+ ImportKind::ExternCrate { id, def_id, .. } => {`
`436`	`435`	`if self.extern_crate_map.get(&def_id).is_none_or(\|&cnum\| {`
`437`	`436`	`!tcx.is_compiler_builtins(cnum)`
`438`	`437`	`&& !tcx.is_panic_runtime(cnum)`
Original file line number	Diff line number	Diff line change
`@@ -2282,8 +2282,7 @@ impl<'ra, 'tcx> Resolver<'ra, 'tcx> {`
`2282`	`2282`
`2283`	`2283`	`#[inline]`
`2284`	`2284`	`fn add_to_glob_map(&mut self, import: Import<'_>, name: Symbol) {`
`2285`		`- if let ImportKind::Glob { id, .. } = import.kind {`
`2286`		`- let def_id = self.local_def_id(id);`
	`2285`	`+ if let ImportKind::Glob { def_id, .. } = import.kind {`
`2287`	`2286`	`self.glob_map.entry(def_id).or_default().insert(name);`
`2288`	`2287`	`}`
`2289`	`2288`	`}`