In #65830 it was discovered that hir::Item.ident.span isn't marked with_ctxt of being part of a macro expansion. This had two effects that we saw there:
- error messages don't include the "in this macro invocation" hints
- cases where the dead_code error was suppressed because of foreign macros were no longer suppressed
To solve that there, we're going to just use the old behavior when we're in the middle of a macro expansion already, so we don't use the un-marked span. As @estebank suggested, we should probably clean this up in a separate change.
In #65830 it was discovered that
hir::Item.ident.spanisn't markedwith_ctxtof being part of a macro expansion. This had two effects that we saw there:To solve that there, we're going to just use the old behavior when we're in the middle of a macro expansion already, so we don't use the un-marked span. As @estebank suggested, we should probably clean this up in a separate change.