Skip to content

Comment about empty run_passes, fixup of #158040#158398

Open
jdonszelmann wants to merge 1 commit into
rust-lang:mainfrom
jdonszelmann:empty_passes
Open

Comment about empty run_passes, fixup of #158040#158398
jdonszelmann wants to merge 1 commit into
rust-lang:mainfrom
jdonszelmann:empty_passes

Conversation

@jdonszelmann

Copy link
Copy Markdown
Contributor

@rustbot

rustbot commented Jun 25, 2026

Copy link
Copy Markdown
Collaborator

Some changes occurred to MIR optimizations

cc @rust-lang/wg-mir-opt

@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 Jun 25, 2026
@rustbot

rustbot commented Jun 25, 2026

Copy link
Copy Markdown
Collaborator

RalfJung is not on the review rotation at the moment.
They may take a while to respond.

// so this would otherwise not get filled).
body.set_mentioned_items(Vec::new());

// We don't pass any passes here, just to force a phase change to `Optimized`.

@RalfJung RalfJung Jun 25, 2026

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.

Suggested change
// We don't pass any passes here, just to force a phase change to `Optimized`.
// We don't pass any passes here, we just force a phase change to `Optimized`.

View changes since the review

@RalfJung

Copy link
Copy Markdown
Member

Thanks!

r=me with grammar/logic nit

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.

3 participants