Skip to content

nr1.0: Remove rust/backend support#3865

Merged
CohenArthur merged 1 commit into
Rust-GCC:masterfrom
powerboat9:rem-back
Jul 4, 2025
Merged

nr1.0: Remove rust/backend support#3865
CohenArthur merged 1 commit into
Rust-GCC:masterfrom
powerboat9:rem-back

Conversation

@powerboat9

Copy link
Copy Markdown
Collaborator

This is the first patch in a set intended to fully remove name resolution 1.0. As such, it should leave name resolution 1.0 technically available but broken.

This is the first patch in a set intended to fully remove name
resolution 1.0. As such, it should leave name resolution 1.0 technically
available but broken.

gcc/rust/ChangeLog:

	* backend/rust-compile-context.cc (Context::Context): Remove
	initialization of resolver field.
	* backend/rust-compile-context.h (Context::get_resolver): Remove
	function.
	(Context::resolver): Remove field.
	* backend/rust-compile-expr.cc (CompileExpr::visit): Assume name
	resolution 2.0 is always enabled.
	(CompileExpr::generate_closure_function): Likewise.
	* backend/rust-compile-implitem.cc (CompileTraitItem::visit):
	Likewise.
	* backend/rust-compile-item.cc (CompileItem::visit): Likewise.
	* backend/rust-compile-resolve-path.cc
	(ResolvePathRef::resolve): Likewise.

Signed-off-by: Owen Avery <powerboat9.gamer@gmail.com>
@powerboat9 powerboat9 changed the title nr1.0: Remove backend support nr1.0: Remove rust/backend support Jul 3, 2025
@powerboat9 powerboat9 changed the title nr1.0: Remove rust/backend support nr1.0: Remove rust/backend support Jul 3, 2025

@CohenArthur CohenArthur left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM! thank you!

@CohenArthur CohenArthur added this pull request to the merge queue Jul 4, 2025
Merged via the queue into Rust-GCC:master with commit a1a56c6 Jul 4, 2025
13 checks passed
@powerboat9 powerboat9 deleted the rem-back branch July 4, 2025 16:38
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