exclude unexported macro bindings from extern crate

rust-lang · Dec 28, 2023 · 10d266b · 10d266b
1 parent 928b3da
commit 10d266b
Show file tree

Hide file tree

Showing 4 changed files with 21 additions and 2 deletions.
diff --git a/compiler/rustc_resolve/src/build_reduced_graph.rs b/compiler/rustc_resolve/src/build_reduced_graph.rs
@@ -1071,7 +1071,8 @@ impl<'a, 'b, 'tcx> BuildReducedGraphVisitor<'a, 'b, 'tcx> {
             let import = macro_use_import(self, span);
             self.r.potentially_unused_imports.push(import);
             module.for_each_child(self, |this, ident, ns, binding| {
-                if ns == MacroNS {
+                if ns == MacroNS && this.r.is_accessible_from(binding.vis, this.parent_scope.module)
+                {
                     let imported_binding = this.r.import(binding, import);
                     this.add_macro_use_binding(ident.name, imported_binding, span, allow_shadowing);
                 }

diff --git a/tests/ui/extern/auxiliary/issue-80074-macro.rs b/tests/ui/extern/auxiliary/issue-80074-macro.rs
@@ -2,3 +2,5 @@
 
 macro_rules! foo_ { () => {}; }
 use foo_ as foo;
+
+macro_rules! bar { () => {}; }
diff --git a/tests/ui/extern/issue-80074.rs b/tests/ui/extern/issue-80074.rs
@@ -1,10 +1,12 @@
 // edition:2018
-// build-pass
 // aux-crate:issue_80074=issue-80074-macro.rs
 
 #[macro_use]
 extern crate issue_80074;
 
 fn main() {
     foo!();
+    //~^ ERROR: cannot find macro `foo` in this scope
+    bar!();
+    //~^ ERROR: cannot find macro `bar` in this scope
 }
diff --git a/tests/ui/extern/issue-80074.stderr b/tests/ui/extern/issue-80074.stderr
@@ -0,0 +1,14 @@
+error: cannot find macro `bar` in this scope
+  --> $DIR/issue-80074.rs:10:5
+   |
+LL |     bar!();
+   |     ^^^
+
+error: cannot find macro `foo` in this scope
+  --> $DIR/issue-80074.rs:8:5
+   |
+LL |     foo!();
+   |     ^^^
+
+error: aborting due to 2 previous errors
+
Original file line number	Diff line number	Diff line change
Expand Up		@@ -2,3 +2,5 @@

		macro_rules! foo_ { () => {}; }
		use foo_ as foo;

		macro_rules! bar { () => {}; }