Skip to content

Use DropCtxt::new_block and new_block_with_statements systematically.#156586

Open
cjgillot wants to merge 3 commits into
rust-lang:mainfrom
cjgillot:elaborate-new-block
Open

Use DropCtxt::new_block and new_block_with_statements systematically.#156586
cjgillot wants to merge 3 commits into
rust-lang:mainfrom
cjgillot:elaborate-new-block

Conversation

@cjgillot
Copy link
Copy Markdown
Contributor

Misc cleanups I found while staring at that code.

@rustbot
Copy link
Copy Markdown
Collaborator

rustbot commented May 14, 2026

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 May 14, 2026
@rustbot
Copy link
Copy Markdown
Collaborator

rustbot commented May 14, 2026

r? @JohnTitor

rustbot has assigned @JohnTitor.
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, mir, mir-opt
  • compiler, mir, mir-opt expanded to 73 candidates
  • Random selection from 17 candidates

Copy link
Copy Markdown
Contributor

@oli-obk oli-obk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@rust-bors
Copy link
Copy Markdown
Contributor

rust-bors Bot commented May 15, 2026

📌 Commit 6e7b089 has been approved by oli-obk

It is now in the queue for this repository.

@rust-bors rust-bors Bot 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 May 15, 2026
@oli-obk oli-obk assigned oli-obk and unassigned JohnTitor May 15, 2026
JonathanBrouwer added a commit to JonathanBrouwer/rust that referenced this pull request May 15, 2026
…i-obk

Use DropCtxt::new_block and new_block_with_statements systematically.

Misc cleanups I found while staring at that code.
rust-bors Bot pushed a commit that referenced this pull request May 15, 2026
…uwer

Rollup of 6 pull requests

Successful merges:

 - #155360 (map_try_insert changes)
 - #156493 (actually run the temp_dir doctest)
 - #156556 (Require UTF-8 in `Utf8Pattern::StringPattern`)
 - #156565 (delegation: emit error when self type is not specified and accessed)
 - #156577 (Test EII UI tests with prefer-dynamic)
 - #156586 (Use DropCtxt::new_block and new_block_with_statements systematically.)
JonathanBrouwer added a commit to JonathanBrouwer/rust that referenced this pull request May 15, 2026
…i-obk

Use DropCtxt::new_block and new_block_with_statements systematically.

Misc cleanups I found while staring at that code.
JonathanBrouwer added a commit to JonathanBrouwer/rust that referenced this pull request May 15, 2026
…i-obk

Use DropCtxt::new_block and new_block_with_statements systematically.

Misc cleanups I found while staring at that code.
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