Skip to content

borrowck: Inline free_region_constraint_info() to simplify the code#158878

Merged
rust-bors[bot] merged 1 commit into
rust-lang:mainfrom
Enselic:inline-free_region_constraint_info
Jul 7, 2026
Merged

borrowck: Inline free_region_constraint_info() to simplify the code#158878
rust-bors[bot] merged 1 commit into
rust-lang:mainfrom
Enselic:inline-free_region_constraint_info

Conversation

@Enselic

@Enselic Enselic commented Jul 6, 2026

Copy link
Copy Markdown
Member

For reference, the helper function was introduced in 2a3969a. But, the code becomes simpler if we remove the helper function, so let's remove it.

This takes us one step closer (this is commit 3 of 4) towards the end result illustrated in #158623.

For reference, the helper function was introduced in 2a3969a. In
any case, the code becomes simpler if we remove it now, so let's remove
it.
@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? @jieyouxu

rustbot has assigned @jieyouxu.
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: borrowck, compiler
  • borrowck, compiler expanded to 75 candidates
  • Random selection from 20 candidates

@jieyouxu jieyouxu left a comment

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.

@rust-bors

rust-bors Bot commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

📌 Commit 690ae7d has been approved by jieyouxu

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 Jul 7, 2026
rust-bors Bot pushed a commit that referenced this pull request Jul 7, 2026
Rollup of 13 pull requests

Successful merges:

 - #158478 (Use correct typing mode in mir building for the next solver)
 - #158796 (Distinguish null reference production from null pointer dereference in UB checks)
 - #158821 (add regression test for late-bound type method probe ICE)
 - #158245 (Update the rustc_perf submodule)
 - #158346 (Prevent rustdoc auto-trait ICEs with `-Znext-solver=globally`)
 - #158536 (Instatiate binder instead of skipping when suggesting function arg error)
 - #158553 (Avoid infinite recursion when trying to build valtrees for self-referential consts)
 - #158679 (Document blocking guarantees for `std::sync`)
 - #158826 (Reorganize `tests/ui/issues` [19/N])
 - #158860 (Delete `impl_def_id` field from `ImplHeader`)
 - #158867 (rewrite `Align::max_for_target` in a more obvious way)
 - #158878 (borrowck: Inline `free_region_constraint_info()` to simplify the code)
 - #158881 (Update mdbook to 0.5.4)
@rust-bors rust-bors Bot merged commit 86de60d into rust-lang:main Jul 7, 2026
13 checks passed
@rustbot rustbot added this to the 1.99.0 milestone Jul 7, 2026
rust-timer added a commit that referenced this pull request Jul 7, 2026
Rollup merge of #158878 - Enselic:inline-free_region_constraint_info, r=jieyouxu

borrowck: Inline `free_region_constraint_info()` to simplify the code

For reference, the helper function was introduced in 2a3969a. But, the code becomes simpler if we remove the helper function, so let's remove it.

This takes us one step closer (this is commit 3 of 4) towards the end result illustrated in #158623.
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.

3 participants