Skip to content

-Zassumptions-on-binders: insert empty assumptions when entering binders in the solver - #157684

Merged
rust-bors[bot] merged 1 commit into
rust-lang:mainfrom
BoxyUwU:empty_assumptions_in_solver_relating
Jun 12, 2026
Merged

-Zassumptions-on-binders: insert empty assumptions when entering binders in the solver#157684
rust-bors[bot] merged 1 commit into
rust-lang:mainfrom
BoxyUwU:empty_assumptions_in_solver_relating

Conversation

@BoxyUwU

@BoxyUwUBoxyUwU commented Jun 9, 2026

Copy link
Copy Markdown
Member

r? lcnr

fixes#157678

computing assumptions in solver relating seems annoying/generally cant move that onto InferCtxtLike because no trait solving is possible there (not that we currently do any when computing assumptions.. thinky.. need to think more :3)

@rustbotrustbot added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. WG-trait-system-refactor The Rustc Trait System Refactor Initiative (-Znext-solver) labels Jun 9, 2026
@rust-log-analyzer

This comment has been minimized.

@lcnrlcnr left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@matthiaskrgr

Copy link
Copy Markdown
Member

cyrgani dropped a more minimal version since the pr was opened:
#157678 (comment)

@BoxyUwU
BoxyUwUforce-pushed the empty_assumptions_in_solver_relating branch from 7d54dc5 to 67cb81eCompareJune 11, 2026 21:51
@BoxyUwU
BoxyUwUforce-pushed the empty_assumptions_in_solver_relating branch from 67cb81e to 390c6a4CompareJune 11, 2026 21:53
@BoxyUwU
BoxyUwU marked this pull request as ready for review June 11, 2026 21:54
@rustbotrustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Jun 11, 2026
@lcnr

lcnr commented Jun 12, 2026

Copy link
Copy Markdown
Contributor

@bors r+ rollup

@rust-bors

rust-borsBot commented Jun 12, 2026

Copy link
Copy Markdown
Contributor

📌 Commit 390c6a4 has been approved by lcnr

It is now in the queue for this repository.

@rust-borsrust-borsBot 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 Jun 12, 2026
GuillaumeGomez added a commit to GuillaumeGomez/rust that referenced this pull request Jun 12, 2026
…r_relating, r=lcnr
-Zassumptions-on-binders: insert empty assumptions when entering binders in the solver
r? lcnr
fixesrust-lang#157678
computing assumptions in solver relating seems annoying/generally cant move that onto `InferCtxtLike` because no trait solving is possible there (not that we currently do any when computing assumptions.. thinky.. need to think more :3)
JonathanBrouwer added a commit to JonathanBrouwer/rust that referenced this pull request Jun 12, 2026
…r_relating, r=lcnr
-Zassumptions-on-binders: insert empty assumptions when entering binders in the solver
r? lcnr
fixesrust-lang#157678
computing assumptions in solver relating seems annoying/generally cant move that onto `InferCtxtLike` because no trait solving is possible there (not that we currently do any when computing assumptions.. thinky.. need to think more :3)
JonathanBrouwer added a commit to JonathanBrouwer/rust that referenced this pull request Jun 12, 2026
…r_relating, r=lcnr
-Zassumptions-on-binders: insert empty assumptions when entering binders in the solver
r? lcnr
fixesrust-lang#157678
computing assumptions in solver relating seems annoying/generally cant move that onto `InferCtxtLike` because no trait solving is possible there (not that we currently do any when computing assumptions.. thinky.. need to think more :3)
rust-borsBot pushed a commit that referenced this pull request Jun 12, 2026
…uwer
Rollup of 17 pull requests
Successful merges:
- #156950 (Staticlib rename internal symbols)
- #157322 (test pre-stabilization items on CI)
- #157490 (Add field-wise CoerceShared reborrow tests)
- #157655 (Make Share::share final and improve docs)
- #157688 (Create experimental test job `aarch64-apple-macos-26` for evaluating `macos-26` runner images)
- #157796 (rustdoc: Some more lazy formatting)
- #157069 (Test that you can't implement Unpin for a compiler-generated future using TAIT)
- #157202 (add #[rustc_no_writable] to slice::get_unchecked_mut)
- #157622 (Disable retagging for variadic arguments in const-eval)
- #157684 (-Zassumptions-on-binders: insert empty assumptions when entering binders in the solver)
- #157695 (Extend capabilities of `TypeFoldable_Generic`)
- #157752 (Rename `errors.rs` file to `diagnostics.rs` (6/N))
- #157766 (interpret: avoid computing layout of sized raw pointee)
- #157785 (fuchsia: Support AddressSanitizer on riscv64gc-unknown-fuchsia)
- #157795 (revert 157013)
- #157798 (Prevent approving PRs that wait for Crater or formal decisions)
- #157803 (Rename `errors.rs` file to `diagnostics.rs` (7/N))
rust-borsBot pushed a commit that referenced this pull request Jun 12, 2026
…uwer
Rollup of 17 pull requests
Successful merges:
- #156950 (Staticlib rename internal symbols)
- #157322 (test pre-stabilization items on CI)
- #157490 (Add field-wise CoerceShared reborrow tests)
- #157655 (Make Share::share final and improve docs)
- #157688 (Create experimental test job `aarch64-apple-macos-26` for evaluating `macos-26` runner images)
- #157796 (rustdoc: Some more lazy formatting)
- #157069 (Test that you can't implement Unpin for a compiler-generated future using TAIT)
- #157202 (add #[rustc_no_writable] to slice::get_unchecked_mut)
- #157622 (Disable retagging for variadic arguments in const-eval)
- #157684 (-Zassumptions-on-binders: insert empty assumptions when entering binders in the solver)
- #157695 (Extend capabilities of `TypeFoldable_Generic`)
- #157752 (Rename `errors.rs` file to `diagnostics.rs` (6/N))
- #157766 (interpret: avoid computing layout of sized raw pointee)
- #157785 (fuchsia: Support AddressSanitizer on riscv64gc-unknown-fuchsia)
- #157795 (revert 157013)
- #157798 (Prevent approving PRs that wait for Crater or formal decisions)
- #157803 (Rename `errors.rs` file to `diagnostics.rs` (7/N))
JonathanBrouwer added a commit to JonathanBrouwer/rust that referenced this pull request Jun 12, 2026
…r_relating, r=lcnr
-Zassumptions-on-binders: insert empty assumptions when entering binders in the solver
r? lcnr
fixesrust-lang#157678
computing assumptions in solver relating seems annoying/generally cant move that onto `InferCtxtLike` because no trait solving is possible there (not that we currently do any when computing assumptions.. thinky.. need to think more :3)
This was referenced Jun 12, 2026
rust-borsBot pushed a commit that referenced this pull request Jun 12, 2026
…uwer
Rollup of 23 pull requests
Successful merges:
- #144220 (Add powerpc64-unknown-linux-gnuelfv2 target)
- #153238 (debuginfo: slices are DW_TAG_array_type's)
- #157112 (Update aarch64-unknown-freebsd target description)
- #157322 (test pre-stabilization items on CI)
- #157348 (Don't track cwd for `-Zremap-cwd-prefix` in incremental compilation)
- #157490 (Add field-wise CoerceShared reborrow tests)
- #157655 (Make Share::share final and improve docs)
- #157672 (Region inference: Simplify initialisation of region values)
- #157680 (Require `#[pin_v2]` for explicit pin-projection patterns)
- #157688 (Create experimental test job `aarch64-apple-macos-26` for evaluating `macos-26` runner images)
- #157796 (rustdoc: Some more lazy formatting)
- #157818 (miri subtree update)
- #157069 (Test that you can't implement Unpin for a compiler-generated future using TAIT)
- #157079 (Don't recover `&raw EXPR` as a missing comma)
- #157202 (add #[rustc_no_writable] to slice::get_unchecked_mut)
- #157622 (Disable retagging for variadic arguments in const-eval)
- #157684 (-Zassumptions-on-binders: insert empty assumptions when entering binders in the solver)
- #157695 (Extend capabilities of `TypeFoldable_Generic`)
- #157766 (interpret: avoid computing layout of sized raw pointee)
- #157785 (fuchsia: Support AddressSanitizer on riscv64gc-unknown-fuchsia)
- #157795 (revert 157013)
- #157798 (Prevent approving PRs that wait for Crater or formal decisions)
- #157803 (Rename `errors.rs` file to `diagnostics.rs` (7/N))
Failed merges:
- #157752 (Rename `errors.rs` file to `diagnostics.rs` (6/N))
@rust-bors
rust-borsBot merged commit 1b6d004 into rust-lang:mainJun 12, 2026
12 checks passed
@rustbotrustbot added this to the 1.98.0 milestone Jun 12, 2026
rust-timer added a commit that referenced this pull request Jun 12, 2026
Rollup merge of #157684 - BoxyUwU:empty_assumptions_in_solver_relating, r=lcnr
-Zassumptions-on-binders: insert empty assumptions when entering binders in the solver
r? lcnr
fixes#157678
computing assumptions in solver relating seems annoying/generally cant move that onto `InferCtxtLike` because no trait solving is possible there (not that we currently do any when computing assumptions.. thinky.. need to think more :3)
pullBot pushed a commit to xtqqczze/rust-lang-miri that referenced this pull request Jun 13, 2026
…uwer
Rollup of 23 pull requests
Successful merges:
- rust-lang/rust#144220 (Add powerpc64-unknown-linux-gnuelfv2 target)
- rust-lang/rust#153238 (debuginfo: slices are DW_TAG_array_type's)
- rust-lang/rust#157112 (Update aarch64-unknown-freebsd target description)
- rust-lang/rust#157322 (test pre-stabilization items on CI)
- rust-lang/rust#157348 (Don't track cwd for `-Zremap-cwd-prefix` in incremental compilation)
- rust-lang/rust#157490 (Add field-wise CoerceShared reborrow tests)
- rust-lang/rust#157655 (Make Share::share final and improve docs)
- rust-lang/rust#157672 (Region inference: Simplify initialisation of region values)
- rust-lang/rust#157680 (Require `#[pin_v2]` for explicit pin-projection patterns)
- rust-lang/rust#157688 (Create experimental test job `aarch64-apple-macos-26` for evaluating `macos-26` runner images)
- rust-lang/rust#157796 (rustdoc: Some more lazy formatting)
- rust-lang/rust#157818 (miri subtree update)
- rust-lang/rust#157069 (Test that you can't implement Unpin for a compiler-generated future using TAIT)
- rust-lang/rust#157079 (Don't recover `&raw EXPR` as a missing comma)
- rust-lang/rust#157202 (add #[rustc_no_writable] to slice::get_unchecked_mut)
- rust-lang/rust#157622 (Disable retagging for variadic arguments in const-eval)
- rust-lang/rust#157684 (-Zassumptions-on-binders: insert empty assumptions when entering binders in the solver)
- rust-lang/rust#157695 (Extend capabilities of `TypeFoldable_Generic`)
- rust-lang/rust#157766 (interpret: avoid computing layout of sized raw pointee)
- rust-lang/rust#157785 (fuchsia: Support AddressSanitizer on riscv64gc-unknown-fuchsia)
- rust-lang/rust#157795 (revert 157013)
- rust-lang/rust#157798 (Prevent approving PRs that wait for Crater or formal decisions)
- rust-lang/rust#157803 (Rename `errors.rs` file to `diagnostics.rs` (7/N))
Failed merges:
- rust-lang/rust#157752 (Rename `errors.rs` file to `diagnostics.rs` (6/N))
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-borsStatus: Waiting on bors to run and complete tests. Bors will change the label on completion.T-compilerRelevant to the compiler team, which will review and decide on the PR/issue.WG-trait-system-refactorThe Rustc Trait System Refactor Initiative (-Znext-solver)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[assumptions-on-binders] unwrap on None ICE in infer/context.rs during mir_borrowck with async closure capturing references

5 participants

@BoxyUwU@rust-log-analyzer@matthiaskrgr@lcnr@rustbot