Skip to content

Conversation

powerboat9
Copy link
Collaborator

Removing this usage of the nr1.0 resolver, without replacing it with a usage of the nr2.0 resolver, seems alright, although I'm not 100% sure about it.

gcc/rust/ChangeLog:

	* expand/rust-macro-expand.cc: Adjust includes.
	(MacroExpander::expand_crate): Remove usage of 1.0 resolver.
	* expand/rust-macro-expand.h: Adjust includes.
	(MacroExpander::MacroExpander): Remove initialization of
	resolver member variable.
	(MacroExpander::resolver): Remove member variable.

Signed-off-by: Owen Avery <powerboat9.gamer@gmail.com>
Copy link
Member

@philberty philberty left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@philberty philberty added this pull request to the merge queue Aug 31, 2025
Merged via the queue into Rust-GCC:master with commit 3e52769 Aug 31, 2025
13 checks passed
@powerboat9 powerboat9 deleted the rem-nr-5 branch August 31, 2025 17:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants