Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

nr2.0: Resolve paths which start with Self #3416

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

nr2.0: Resolve paths which start with Self #3416

wants to merge 1 commit into from

Conversation

powerboat9
Copy link
Collaborator

Overlaps with #3391 (apologies to @P-E-P, but this should be a more general resolution), depends on #3220

@P-E-P
Copy link
Member

P-E-P commented Feb 4, 2025

Overlaps with #3391 (apologies to @P-E-P, but this should be a more general resolution), depends on #3220

I'm kinda glad you came up with another solution because mine felt wrong and I was even thinking about starting it from scratch again.

@P-E-P P-E-P self-requested a review February 4, 2025 10:56
@powerboat9 powerboat9 mentioned this pull request Feb 4, 2025
Open
@powerboat9
nr2.0: Resolve paths which start with Self
e9262a5 
gcc/rust/ChangeLog:

	* resolve/rust-forever-stack.hxx
	(ForeverStack::find_starting_point): Be more careful about
	applying ForeverStack::find_closest_module.
	(ForeverStack::resolve_segments): Allow traversal into parent
	nodes when not in a module node or root node, which
	ForeverStack::find_starting_point previously made moot through
	use of ForeverStack::find_closest_module. Also, when a child
	node lookup fails when resolving in the type namespace, attempt
	a rib lookup as a fallback.
	* resolve/rust-late-name-resolver-2.0.cc
	(Late::visit): Avoid throwing a resolution error for type paths
	when the typechecker may be able to finish the resolution. Also,
	throw an error when a resolution is ambiguous.

gcc/testsuite/ChangeLog:

	* rust/compile/nr2/exclude: Remove entries.

Signed-off-by: Owen Avery <[email protected]>
@P-E-P P-E-P added this pull request to the merge queue Feb 5, 2025
@P-E-P P-E-P removed this pull request from the merge queue due to a manual request Feb 5, 2025
@P-E-P P-E-P requested a review from CohenArthur February 5, 2025 13:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@P-E-P P-E-P P-E-P approved these changes

@CohenArthur CohenArthur Awaiting requested review from CohenArthur

Assignees
No one assigned
Labels
name resolution
Projects
name resolution 2.0
Status: In Progress
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@powerboat9 @P-E-P