Skip to content

Avoid final override ICE for RPITIT associated types#158841

Open
TaKO8Ki wants to merge 2 commits into
rust-lang:mainfrom
TaKO8Ki:fix-final-rpitit-override-ice
Open

Avoid final override ICE for RPITIT associated types#158841
TaKO8Ki wants to merge 2 commits into
rust-lang:mainfrom
TaKO8Ki:fix-final-rpitit-override-ice

Conversation

@TaKO8Ki

@TaKO8Ki TaKO8Ki commented Jul 6, 2026

Copy link
Copy Markdown
Member

Fixes #158824

check_overriding_final_trait_item was running for every associated item in an impl, including the synthetic anonymous associated type generated for RPITIT.

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. labels Jul 6, 2026
@rustbot

rustbot commented Jul 6, 2026

Copy link
Copy Markdown
Collaborator

r? @mejrs

rustbot has assigned @mejrs.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

Why was this reviewer chosen?

The reviewer was selected based on:

  • Owners of files modified in this PR: compiler
  • compiler expanded to 75 candidates
  • Random selection from 20 candidates

@mejrs

mejrs commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

Does this mean that rpitit's inherit the defaultness of the function that defines it?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[ICE]: item_ident: no name for DefPath

3 participants