resolve: When suppressing out_of_scope_macro_calls suppress unused_imports as well

Author: petrochenkov

compiler/rustc_resolve/src/macros.rs

@@ -1107,16 +1107,22 @@ impl<'ra, 'tcx> Resolver<'ra, 'tcx> {
             // If such resolution is successful and gives the same result
             // (e.g. if the macro is re-imported), then silence the lint.
             let no_macro_rules = self.arenas.alloc_macro_rules_scope(MacroRulesScope::Empty);
+            let ident = path.segments[0].ident;
             let fallback_binding = self.reborrow().resolve_ident_in_scope_set(
-                path.segments[0].ident,
+                ident,
                 ScopeSet::Macro(MacroKind::Bang),
                 &ParentScope { macro_rules: no_macro_rules, ..*parent_scope },
                 None,
                 false,
                 None,
                 None,
             );
-            if fallback_binding.ok().and_then(|b| b.res().opt_def_id()) != Some(def_id) {
+            if let Ok(fallback_binding) = fallback_binding
+                && fallback_binding.res().opt_def_id() == Some(def_id)
+            {
+                // Silence `unused_imports` on the fallback import as well.
+                self.get_mut().record_use(ident, fallback_binding, Used::Other);
+            } else {
                 let location = match parent_scope.module.kind {
                     ModuleKind::Def(kind, def_id, name) => {
                         if let Some(name) = name {

tests/ui/attributes/key-value-expansion-scope-pass.rs

@@ -3,6 +3,7 @@
 //@ check-pass
 //@ edition:2018
 
+#![warn(unused_imports)]
 #![doc = in_root!()]
 
 macro_rules! in_root { () => { "" } }