Skip to content

Conversation

powerboat9
Copy link
Collaborator

No description provided.

@powerboat9
Copy link
Collaborator Author

I've used X macros to replace the lambdas

gcc/rust/ChangeLog:

	* Make-lang.in (GRS_OBJS): Add "rust-resolve-builtins.o".
	* resolve/rust-late-name-resolver-2.0.cc: Include
	"rust-resolve-builtins.h".
	(next_node_id): Remove function.
	(next_hir_id): Likewise.
	(Late::setup_builtin_types): Likewise.
	(Late::go): Use Builtins::setup_type_ctx instead of
	Late::setup_builtin_types.
	* resolve/rust-late-name-resolver-2.0.h
	(Late::setup_builtin_types): Remove function.
	* rust-session-manager.cc: Include "rust-resolve-builtins.h".
	(Session::expansion): Call Builtins::setup_lang_prelude.
	* resolve/rust-resolve-builtins.cc: New file.
	* resolve/rust-resolve-builtins.h: New file.

gcc/testsuite/ChangeLog:

	* rust/compile/primitive-import.rs: New test.

Signed-off-by: Owen Avery <[email protected]>
@powerboat9
Copy link
Collaborator Author

@CohenArthur thoughts on merging?

Copy link
Member

@CohenArthur CohenArthur left a comment

Choose a reason for hiding this comment

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

That looks good to me, thanks for working on this Owen!

@CohenArthur CohenArthur added this pull request to the merge queue Aug 26, 2025
Merged via the queue into Rust-GCC:master with commit 522d2d3 Aug 26, 2025
13 checks passed
@powerboat9 powerboat9 deleted the fix-lang-import branch August 26, 2025 22:18
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