exclude unexported macro bindings from extern crate

bvanjoi · bvanjoi · commit 6b62275bc9e6 · 2023-12-28T23:35:26.000+08:00
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-2.rs b/tests/ui/extern/auxiliary/issue-80074-macro-2.rs
@@ -0,0 +1,3 @@
+// edition:2018
+
+macro_rules! boo { () => {}; }
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,19 @@
 // edition:2018
-// build-pass
 // aux-crate:issue_80074=issue-80074-macro.rs
+// aux-crate:issue_80074_2=issue-80074-macro-2.rs
 
 #[macro_use]
 extern crate issue_80074;
 
+#[macro_use(boo)]
+extern crate issue_80074_2;
+//~^^ ERROR: imported macro not found
+
 fn main() {
     foo!();
+    //~^ ERROR: cannot find macro `foo` in this scope
+    bar!();
+    //~^ ERROR: cannot find macro `bar` in this scope
+    boo!();
+    //~^ ERROR: cannot find macro `boo` in this scope
 }
diff --git a/tests/ui/extern/issue-80074.stderr b/tests/ui/extern/issue-80074.stderr
@@ -0,0 +1,27 @@
+error[E0469]: imported macro not found
+  --> $DIR/issue-80074.rs:8:13
+   |
+LL | #[macro_use(boo)]
+   |             ^^^
+
+error: cannot find macro `boo` in this scope
+  --> $DIR/issue-80074.rs:17:5
+   |
+LL |     boo!();
+   |     ^^^
+
+error: cannot find macro `bar` in this scope
+  --> $DIR/issue-80074.rs:15:5
+   |
+LL |     bar!();
+   |     ^^^
+
+error: cannot find macro `foo` in this scope
+  --> $DIR/issue-80074.rs:13:5
+   |
+LL |     foo!();
+   |     ^^^
+
+error: aborting due to 4 previous errors
+
+For more information about this error, try `rustc --explain E0469`.

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,3 @@`
	`1`	`+// edition:2018`
	`2`	`+`
	`3`	`+macro_rules! boo { () => {}; }`