Repository navigation
Conversation
This comment has been minimized.
This comment has been minimized.
c3ef9cd to
f631ca8
Compare
|
@bors try |
[DO NOT MERGE] bootstrap with `-Znext-solver=globally` A revival of rust-lang#124812. Current status, we're failing in: - failing in `rustc_next_trait_solver` with 126 instances of the following error ``` error[E0311]: the parameter type `I` may not live long enough | help: consider adding an explicit lifetime bound --> compiler/rustc_next_trait_solver/src/solve/trait_goals.rs:624:53 | 624 ~ fn consider_structural_builtin_unsize_candidates<'a>( 625 ~ ecx: &mut EvalCtxt<'a, D>, 626 | goal: Goal<I, Self>, 627 ~ ) -> Vec<Candidate<I>> where I: 'a { ``` - `itertools` hangs, we're already encountered this hang in our previous attempt due to large type sizes. I believe that it's simply caused by a missing cache somewhere, potentially in `wf.rs`, but other visitors may also be responsible. See rust-itertools/itertools#945 for more details ### commits - rust-lang#133501 - rust-lang#133493 - 9456bfe and b21b116 reimplement candidate preference based on rust-lang#132325, not yet a separate PR - c3ef9cd is a rebased version of rust-lang#125334, unsure whether I actually want to land this PR for now r? `@ghost`
|
I also pushed:
I think this is sufficient to make the compiler build, or at least |
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment has been minimized.
This comment has been minimized.
|
💔 Test failed - checks-actions |
|
☔ The latest upstream changes (presumably #133527) made this pull request unmergeable. Please resolve the merge conflicts. |
|
Next failure is on pub fn repro() -> impl FnMut() {
if true {
|| ()
} else {
|| ()
}
} |
f631ca8 to
6210ed3
Compare
|
@bors try |
[DO NOT MERGE] bootstrap with `-Znext-solver=globally` A revival of rust-lang#124812. Current status: `./x.py b --stage 2` passes 🎉 ### commits - rust-lang#133501 - rust-lang#133493 - 9456bfe and b21b116 reimplement candidate preference based on rust-lang#132325, not yet a separate PR - c3ef9cd is a rebased version of rust-lang#125334, unsure whether I actually want to land this PR for now - rust-lang#133517 * rust-lang#133518 * rust-lang#133519 * rust-lang#133520 * rust-lang#133521 * rust-lang#133524 r? `@ghost`
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
|
💔 Test failed - checks-actions |
6210ed3 to
8fa8f40
Compare
|
@bors try |
[DO NOT MERGE] bootstrap with `-Znext-solver=globally` A revival of rust-lang#124812. Current status: `./x.py b --stage 2` passes 🎉 ### commits - rust-lang#133501 - rust-lang#133493 - 9456bfe and b21b116 reimplement candidate preference based on rust-lang#132325, not yet a separate PR - c3ef9cd is a rebased version of rust-lang#125334, unsure whether I actually want to land this PR for now - rust-lang#133517 * rust-lang#133518 * rust-lang#133519 * rust-lang#133520 * rust-lang#133521 * rust-lang#133524 r? `@ghost`
|
I was reducing the new failure but you've already fixed it ^^ |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
|
Partially because I genuinely think getting the next solver stabilized asap is very useful as it blocks a lot of other work. partially because I am slowly going insane, and the idea of being stuck while waiting on crater is :<< @craterbot p=6 |
|
📝 Configuration of the ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more |
This comment has been minimized.
This comment has been minimized.
|
🚧 Experiment ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more |
|
🎉 Experiment
Footnotes
|
|
@craterbot check p=6 crates=https://crater-reports.s3.amazonaws.com/pr-133502-29/retry-regressed-list.txt (Mirroring the priority from the original run.) |
|
👌 Experiment ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more |
|
@craterbot p=1 (Looking at the queue, there are things pending for the release, and it seems better to not get between those.) |
|
📝 Configuration of the ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more |
|
📝 Configuration of the ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more |
|
something's wrong 🤔 that's too few failures 😅 we should have minijinja failing @craterbot cancel |
|
🗑️ Experiment ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more |
|
see https://rust-lang.zulipchat.com/#narrow/channel/364551-t-types.2Ftrait-system-refactor/topic/crater.20triage.20.3A3/near/629752801. we're comparing with nightly. nightly already has the new solver enabled. This is effectively a crater run of just #163589. There should be a way to only run crater for failing crates + spurious regressions |
|
@bors try |
This comment has been minimized.
This comment has been minimized.
|
filtered the crates which also failed in the start run of #133502 (comment) @craterbot run mode=check-only start=cc155fe68aae7ab0c83c7a5197b35ba12ec13c92+rustflags=-Znext-solver=coherence end=cc155fe68aae7ab0c83c7a5197b35ba12ec13c92 crates=https://gist.githubusercontent.com/lcnr/0e2a09bb6b565ee9392c664a3c8cb47d/raw/a7aee74336eeb45df44563106cc3c9fc5930eece/failed-crates.txt p=6 |
|
👌 Experiment ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more |
|
we've got time, gonna fix the issues from #163589 first @craterbot cancel :> |
|
🚨 Error: failed to parse the command 🆘 If you have any trouble with Crater please ask in t-infra on Zulip |
|
@bors try |
|
⌛ Trying commit df5e3b5 with merge df71ca4… To cancel the try build, run the command Workflow: https://github.com/rust-lang/rust/actions/runs/37609720996 |
|
@craterbot cancel |
|
🗑️ Experiment ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more |
|
@craterbot run mode=check-only start=df71ca4bba430db14e784e45bb5259340e88b3ad+rustflags=-Znext-solver=coherence end=df71ca4bba430db14e784e45bb5259340e88b3ad crates=https://gist.githubusercontent.com/lcnr/0e2a09bb6b565ee9392c664a3c8cb47d/raw/a7aee74336eeb45df44563106cc3c9fc5930eece/failed-crates.txt p=6 |
|
👌 Experiment ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more |

View all comments
A revival of #124812. See
-Znext-solver#107374Current status:
./x.py b --stage 2passes 🎉trybuilds succeed 🎉 🎉 🎉first perf run 👻 second perf run 👻 third perf run 👻 fourth perf run 👻 fifth perf run 👻
crater
This does not detect hangs or memory issues. We are triaging the regressions in https://rust-lang.zulipchat.com/#narrow/channel/364551-t-types.2Ftrait-system-refactor/topic/crater.20triage.20.3A3/with/572783267 and welcome your help there.
in-flight changes
-Znext-solver#163928r? @ghost
Footnotes
a complete crater run ↩ ↩2 ↩3 ↩4 ↩5
with doubled
recursion_limitwhen evaluating goals ↩ ↩2 ↩3 ↩4 ↩5 ↩6 ↩7 ↩8 ↩9 ↩10 ↩11 ↩12only testing crates which may have regressed from the above run ↩ ↩2 ↩3 ↩4 ↩5 ↩6 ↩7 ↩8 ↩9 ↩10 ↩11 ↩12