Skip to content

GVN: Do not flatten derefs with ProjectionElem::Index. #145030

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.

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

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

cjgillot
Copy link
Contributor

@cjgillot cjgillot commented Aug 7, 2025

r? @saethlin

This should fix the bug you found with #131650

@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 Aug 7, 2025
@rustbot
Copy link
Collaborator

rustbot commented Aug 7, 2025

Some changes occurred to MIR optimizations

cc @rust-lang/wg-mir-opt

@saethlin
Copy link
Member

saethlin commented Aug 7, 2025

Oof, that's subtle. Thanks for fixing this so quickly.

@bors r+

@bors
Copy link
Collaborator

bors commented Aug 7, 2025

📌 Commit 0f7f574 has been approved by saethlin

It is now in the queue for this repository.

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Aug 7, 2025
@cjgillot cjgillot force-pushed the gvn-no-flatten-index branch from 0f7f574 to ebd60b9 Compare August 7, 2025 23:34
@cjgillot
Copy link
Contributor Author

cjgillot commented Aug 7, 2025

Added a missing mir-opt test output.
@bors r=saethlin

@bors
Copy link
Collaborator

bors commented Aug 7, 2025

📌 Commit ebd60b9 has been approved by saethlin

It is now in the queue for this repository.

Zalathar added a commit to Zalathar/rust that referenced this pull request Aug 8, 2025
…aethlin

GVN:  Do not flatten derefs with ProjectionElem::Index.

r? `@saethlin`

This should fix the bug you found with rust-lang#131650
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. 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.

4 participants