Skip to content

gccrs: Fix silly ordering bug in trait reference resolution#3844

Merged
philberty merged 1 commit into
masterfrom
phil/fix-ordering-bug
Jun 22, 2025
Merged

gccrs: Fix silly ordering bug in trait reference resolution#3844
philberty merged 1 commit into
masterfrom
phil/fix-ordering-bug

Conversation

@philberty

Copy link
Copy Markdown
Member

Ensure proper ordering when resolving trait references to prevent incorrect type resolution in certain contexts.

gcc/rust/ChangeLog:

* typecheck/rust-hir-trait-reference.cc: Fix ordering bug.
* typecheck/rust-hir-type-check-type.cc: Update call site.
* testsuite/rust/compile/silly-order-bug.rs: New test.

Ensure proper ordering when resolving trait references to prevent
incorrect type resolution in certain contexts.

gcc/rust/ChangeLog:

	* typecheck/rust-hir-trait-reference.cc (TraitReference::on_resolved): ensure associated
	types are done first
	* typecheck/rust-hir-type-check-type.cc: Update call site.

gcc/testsuite/ChangeLog:

	* rust/compile/silly-order-bug.rs: New test.

Signed-off-by: Philip Herron <herron.philip@googlemail.com>
@philberty philberty force-pushed the phil/fix-ordering-bug branch from 7bce4cf to 8d21d80 Compare June 22, 2025 20:50
@philberty philberty added this pull request to the merge queue Jun 22, 2025
Merged via the queue into master with commit eb33ed9 Jun 22, 2025
12 checks passed
@github-project-automation github-project-automation Bot moved this from Todo to Done in libcore 1.49 Jun 22, 2025
@philberty philberty deleted the phil/fix-ordering-bug branch June 23, 2025 10:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant