Skip to content

even more tests/crashes migration for -Znext-solver - #163806

Merged
rust-bors[bot] merged 27 commits into
rust-lang:mainfrom
lcnr:crashes-migration-cont
Oct 8, 2026
Merged

rust-bors[bot] merged 27 commits into
rust-lang:mainfrom
lcnr:crashes-migration-cont

Conversation

@lcnr

@lcnr lcnr commented Oct 5, 2026 •

Copy link
Copy Markdown
Contributor

cc https://hackmd.io/ydc_nIbHQqG-ctMDapXYTg continuing the work of #163796

relevant issues:

@rustbot rustbot 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. labels Oct 5, 2026
@lcnr lcnr changed the title even more crashes migration for -Znext-solver even more tests/crashes migration for -Znext-solver Oct 5, 2026
@rust-log-analyzer

This comment has been minimized.

@lcnr
lcnr force-pushed the crashes-migration-cont branch 2 times, most recently from 03239a7 to 5746876 Compare October 5, 2026 13:50
@rustbot rustbot added the WG-trait-system-refactor The Rustc Trait System Refactor Initiative (-Znext-solver) label Oct 5, 2026
@lcnr
lcnr force-pushed the crashes-migration-cont branch 2 times, most recently from 03f305a to 5d2cae8 Compare October 5, 2026 21:26
@rust-log-analyzer

This comment has been minimized.

@lcnr

lcnr commented Oct 6, 2026

Copy link
Copy Markdown
Contributor Author

r? lqd i guess :3

@rustbot

rustbot commented Oct 6, 2026

Copy link
Copy Markdown
Collaborator

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

@lcnr
lcnr marked this pull request as ready for review October 6, 2026 09:45
@rustbot

rustbot commented Oct 6, 2026

Copy link
Copy Markdown
Collaborator

This PR changes a file inside tests/crashes. If a crash was fixed, please move into the corresponding ui subdir and add 'Fixes #' to the PR description to autoclose the issue upon merge.

@rustbot rustbot 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 Oct 6, 2026

@lqd lqd 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.

//@[old] known-bug: #135617

// The bound `for<'a> (): Project` prevents us from normalizing the
// array length. This caused and ICE with the new solver. The actual

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.

Did you mean "This caused an ICE with the old solver"?

Comment on lines +3 to +7
//@[next] compile-flags: -Znext-solver
//@[old] failure-status: 101
//@[old] dont-check-compiler-stderr
//@[old] known-bug: #140850
//@[next] check-pass

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.

(if we end up modifying anything in this PR, let's also regroup the two next directives together so it's slightly easier to read)

Comment on lines +3 to +7
//@[next] compile-flags: -Znext-solver
//@[old] failure-status: 101
//@[old] dont-check-compiler-stderr
//@[old] known-bug: #125553
//@[next] check-pass

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.

(here as well, if we end up modifying anything in this PR, let's also regroup the two next directives together so it's slightly easier to read)

@lcnr
lcnr force-pushed the crashes-migration-cont branch from f4b8aae to 579a6bd Compare October 6, 2026 15:46
@rustbot

rustbot commented Oct 6, 2026

Copy link
Copy Markdown
Collaborator

This PR was rebased onto a different main commit. Here's a range-diff highlighting what actually changed.

Rebasing is a normal part of keeping PRs up to date, so no action is needed—this note is just to help reviewers.

@lcnr

lcnr commented Oct 6, 2026

Copy link
Copy Markdown
Contributor Author

@bors r=lqd rollup=iffy (lots of test, otherwise safe)

@rust-bors

rust-bors Bot commented Oct 6, 2026

Copy link
Copy Markdown
Contributor

📌 Commit 579a6bd has been tentatively approved by lqd

It will be put into the queue for this repository once PR CI succeeds.

@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 Oct 6, 2026
rust-bors Bot pushed a commit that referenced this pull request Oct 7, 2026
…uwer

Rollup of 25 pull requests

Successful merges:

 - #163090 (Run LLDB debuginfo tests on `x86_64-pc-windows-msvc` in CI)
 - #163806 (even more `tests/crashes` migration for `-Znext-solver`)
 - #162156 (add IBM f128 type)
 - #163508 (Document the `rustc_on_unimplemented` attribute.)
 - #163794 (Use `target_family = "wasm"` instead of `target_arch = "wasm32"`)
 - #163863 (use the type name instead of `Self` in constructor paths again)
 - #163864 (Remove CLAUDE.md)
 - #163877 (Add regression test for duplicated rustdoc search results between std and core)
 - #163916 (Shrink `PartialRes` and its alignment)
 - #149753 (On name resolution error in parameter list, suggest possible `const` typo and avoid unnecessary second error)
 - #152557 (Oneshot `is_ready`)
 - #157273 (Stabilize `optimize` attribute)
 - #162000 (peel_transparent_wrappers only works on non-1ZST)
 - #163257 (abby DSL: sanity checks on forall where clauses)
 - #163343 (`unit` and `[1 x unit]` layouts are incompatible on big-endian targets)
 - #163348 (perf:cache FFI safety results in improper_ctypes lint)
 - #163768 (Move more `rustdoc-html` tests in the right location)
 - #163778 (check is_indirect_first_projection when replacing in RefProp)
 - #163789 (cg_llvm: Avoid `as_c_char_ptr` in several places)
 - #163807 (Add `has_reliable_f16b` for Arm)
 - #163809 (Switch TLS implementation for wasi and bump SDK version to 34)
 - #163826 (Update books)
 - #163849 (ci: update to PowerShell 7.6.6)
 - #163869 (INSTALL.md: use UCRT64 instead of MINGW64 in MSYS2 section)
 - #163870 (Add if-installed:auto:spellcheck to pre-push script)
rust-bors Bot pushed a commit that referenced this pull request Oct 7, 2026
…uwer

Rollup of 24 pull requests

Successful merges:

 - #163090 (Run LLDB debuginfo tests on `x86_64-pc-windows-msvc` in CI)
 - #163806 (even more `tests/crashes` migration for `-Znext-solver`)
 - #163508 (Document the `rustc_on_unimplemented` attribute.)
 - #163794 (Use `target_family = "wasm"` instead of `target_arch = "wasm32"`)
 - #163863 (use the type name instead of `Self` in constructor paths again)
 - #163864 (Remove CLAUDE.md)
 - #163877 (Add regression test for duplicated rustdoc search results between std and core)
 - #163916 (Shrink `PartialRes` and its alignment)
 - #149753 (On name resolution error in parameter list, suggest possible `const` typo and avoid unnecessary second error)
 - #152557 (Oneshot `is_ready`)
 - #157273 (Stabilize `optimize` attribute)
 - #162000 (peel_transparent_wrappers only works on non-1ZST)
 - #163257 (abby DSL: sanity checks on forall where clauses)
 - #163343 (`unit` and `[1 x unit]` layouts are incompatible on big-endian targets)
 - #163348 (perf:cache FFI safety results in improper_ctypes lint)
 - #163768 (Move more `rustdoc-html` tests in the right location)
 - #163778 (check is_indirect_first_projection when replacing in RefProp)
 - #163789 (cg_llvm: Avoid `as_c_char_ptr` in several places)
 - #163807 (Add `has_reliable_f16b` for Arm)
 - #163809 (Switch TLS implementation for wasi and bump SDK version to 34)
 - #163826 (Update books)
 - #163849 (ci: update to PowerShell 7.6.6)
 - #163869 (INSTALL.md: use UCRT64 instead of MINGW64 in MSYS2 section)
 - #163870 (Add if-installed:auto:spellcheck to pre-push script)
rust-bors Bot pushed a commit that referenced this pull request Oct 7, 2026
…uwer

Rollup of 24 pull requests

Successful merges:

 - #163090 (Run LLDB debuginfo tests on `x86_64-pc-windows-msvc` in CI)
 - #163806 (even more `tests/crashes` migration for `-Znext-solver`)
 - #163508 (Document the `rustc_on_unimplemented` attribute.)
 - #163794 (Use `target_family = "wasm"` instead of `target_arch = "wasm32"`)
 - #163863 (use the type name instead of `Self` in constructor paths again)
 - #163864 (Remove CLAUDE.md)
 - #163877 (Add regression test for duplicated rustdoc search results between std and core)
 - #163916 (Shrink `PartialRes` and its alignment)
 - #149753 (On name resolution error in parameter list, suggest possible `const` typo and avoid unnecessary second error)
 - #152557 (Oneshot `is_ready`)
 - #157273 (Stabilize `optimize` attribute)
 - #162000 (peel_transparent_wrappers only works on non-1ZST)
 - #163257 (abby DSL: sanity checks on forall where clauses)
 - #163343 (`unit` and `[1 x unit]` layouts are incompatible on big-endian targets)
 - #163348 (perf:cache FFI safety results in improper_ctypes lint)
 - #163768 (Move more `rustdoc-html` tests in the right location)
 - #163778 (check is_indirect_first_projection when replacing in RefProp)
 - #163789 (cg_llvm: Avoid `as_c_char_ptr` in several places)
 - #163807 (Add `has_reliable_f16b` for Arm)
 - #163809 (Switch TLS implementation for wasi and bump SDK version to 34)
 - #163826 (Update books)
 - #163849 (ci: update to PowerShell 7.6.6)
 - #163869 (INSTALL.md: use UCRT64 instead of MINGW64 in MSYS2 section)
 - #163870 (Add if-installed:auto:spellcheck to pre-push script)
rust-bors Bot pushed a commit that referenced this pull request Oct 7, 2026
…uwer

Rollup of 24 pull requests

Successful merges:

 - #163090 (Run LLDB debuginfo tests on `x86_64-pc-windows-msvc` in CI)
 - #163806 (even more `tests/crashes` migration for `-Znext-solver`)
 - #163508 (Document the `rustc_on_unimplemented` attribute.)
 - #163794 (Use `target_family = "wasm"` instead of `target_arch = "wasm32"`)
 - #163863 (use the type name instead of `Self` in constructor paths again)
 - #163864 (Remove CLAUDE.md)
 - #163877 (Add regression test for duplicated rustdoc search results between std and core)
 - #163916 (Shrink `PartialRes` and its alignment)
 - #149753 (On name resolution error in parameter list, suggest possible `const` typo and avoid unnecessary second error)
 - #152557 (Oneshot `is_ready`)
 - #157273 (Stabilize `optimize` attribute)
 - #162000 (peel_transparent_wrappers only works on non-1ZST)
 - #163257 (abby DSL: sanity checks on forall where clauses)
 - #163343 (`unit` and `[1 x unit]` layouts are incompatible on big-endian targets)
 - #163348 (perf:cache FFI safety results in improper_ctypes lint)
 - #163768 (Move more `rustdoc-html` tests in the right location)
 - #163778 (check is_indirect_first_projection when replacing in RefProp)
 - #163789 (cg_llvm: Avoid `as_c_char_ptr` in several places)
 - #163807 (Add `has_reliable_f16b` for Arm)
 - #163809 (Switch TLS implementation for wasi and bump SDK version to 34)
 - #163826 (Update books)
 - #163849 (ci: update to PowerShell 7.6.6)
 - #163869 (INSTALL.md: use UCRT64 instead of MINGW64 in MSYS2 section)
 - #163870 (Add if-installed:auto:spellcheck to pre-push script)
rust-bors Bot pushed a commit that referenced this pull request Oct 7, 2026
…uwer

Rollup of 24 pull requests

Successful merges:

 - #163090 (Run LLDB debuginfo tests on `x86_64-pc-windows-msvc` in CI)
 - #163806 (even more `tests/crashes` migration for `-Znext-solver`)
 - #163508 (Document the `rustc_on_unimplemented` attribute.)
 - #163794 (Use `target_family = "wasm"` instead of `target_arch = "wasm32"`)
 - #163863 (use the type name instead of `Self` in constructor paths again)
 - #163864 (Remove CLAUDE.md)
 - #163877 (Add regression test for duplicated rustdoc search results between std and core)
 - #163916 (Shrink `PartialRes` and its alignment)
 - #149753 (On name resolution error in parameter list, suggest possible `const` typo and avoid unnecessary second error)
 - #152557 (Oneshot `is_ready`)
 - #157273 (Stabilize `optimize` attribute)
 - #162000 (peel_transparent_wrappers only works on non-1ZST)
 - #163257 (abby DSL: sanity checks on forall where clauses)
 - #163343 (`unit` and `[1 x unit]` layouts are incompatible on big-endian targets)
 - #163348 (perf:cache FFI safety results in improper_ctypes lint)
 - #163768 (Move more `rustdoc-html` tests in the right location)
 - #163778 (check is_indirect_first_projection when replacing in RefProp)
 - #163789 (cg_llvm: Avoid `as_c_char_ptr` in several places)
 - #163807 (Add `has_reliable_f16b` for Arm)
 - #163809 (Switch TLS implementation for wasi and bump SDK version to 34)
 - #163826 (Update books)
 - #163849 (ci: update to PowerShell 7.6.6)
 - #163869 (INSTALL.md: use UCRT64 instead of MINGW64 in MSYS2 section)
 - #163870 (Add if-installed:auto:spellcheck to pre-push script)
rust-bors Bot pushed a commit that referenced this pull request Oct 7, 2026
…uwer

Rollup of 23 pull requests

Successful merges:

 - #163090 (Run LLDB debuginfo tests on `x86_64-pc-windows-msvc` in CI)
 - #163806 (even more `tests/crashes` migration for `-Znext-solver`)
 - #163508 (Document the `rustc_on_unimplemented` attribute.)
 - #163794 (Use `target_family = "wasm"` instead of `target_arch = "wasm32"`)
 - #163863 (use the type name instead of `Self` in constructor paths again)
 - #163864 (Remove CLAUDE.md)
 - #163877 (Add regression test for duplicated rustdoc search results between std and core)
 - #163916 (Shrink `PartialRes` and its alignment)
 - #149753 (On name resolution error in parameter list, suggest possible `const` typo and avoid unnecessary second error)
 - #152557 (Oneshot `is_ready`)
 - #157273 (Stabilize `optimize` attribute)
 - #162000 (peel_transparent_wrappers only works on non-1ZST)
 - #163343 (`unit` and `[1 x unit]` layouts are incompatible on big-endian targets)
 - #163348 (perf:cache FFI safety results in improper_ctypes lint)
 - #163768 (Move more `rustdoc-html` tests in the right location)
 - #163778 (check is_indirect_first_projection when replacing in RefProp)
 - #163789 (cg_llvm: Avoid `as_c_char_ptr` in several places)
 - #163807 (Add `has_reliable_f16b` for Arm)
 - #163809 (Switch TLS implementation for wasi and bump SDK version to 34)
 - #163826 (Update books)
 - #163849 (ci: update to PowerShell 7.6.6)
 - #163869 (INSTALL.md: use UCRT64 instead of MINGW64 in MSYS2 section)
 - #163870 (Add if-installed:auto:spellcheck to pre-push script)
@rust-bors
rust-bors Bot merged commit 1760e75 into rust-lang:main Oct 8, 2026
14 checks passed
@rustbot rustbot added this to the 1.101.0 milestone Oct 8, 2026
rust-bors Bot pushed a commit that referenced this pull request Oct 8, 2026
Rollup merge of #163806 - lcnr:crashes-migration-cont, r=lqd

even more `tests/crashes` migration for `-Znext-solver`

cc https://hackmd.io/ydc_nIbHQqG-ctMDapXYTg continuing the work of #163796

relevant issues:
- #126680 still broken due to rust-lang/trait-system-refactor-initiative#303
- #140850
- #125553
- #123157 duplicate of #125553
- #131886 still broken due to rust-lang/trait-system-refactor-initiative#303
- #135617
- #129372
- #134838
- #131373
- #136666 duplicate of #124440
- #127033
- #130797 duplicate of #140211
- #135122
- #135718
- #139381
- #138707
- #130524 duplicate of #140211
- #147719 instance of #110534
- #148094 duplicate of #131886
- #139120 duplicate of #140211
- #146965 instance of lcnr/random-rust-snippets#23
- #149015
- #148095
- #150263 duplicate of #146965
- #150517 duplicate of #149015
- #151069
@rust-timer

Copy link
Copy Markdown
Collaborator

Note

This PR was benchmarked as part of triage of its containing rollup: triage URL.

Finished benchmarking commit (169ad55): comparison URL.

Overall result: ✅ improvements - no action needed

@rustbot label: -perf-regression

Instruction count

Our most reliable metric. Used to determine the overall result above. However, even this metric can be noisy.

mean range count
Regressions ❌
(primary)
- - 0
Regressions ❌
(secondary)
- - 0
Improvements ✅
(primary)
-0.1% [-0.1%, -0.1%] 2
Improvements ✅
(secondary)
-0.1% [-0.1%, -0.1%] 6
All ❌✅ (primary) -0.1% [-0.1%, -0.1%] 2

Max RSS (memory usage)

Results (secondary -2.1%)

A less reliable metric. May be of interest, but not used to determine the overall result above.

mean range count
Regressions ❌
(primary)
- - 0
Regressions ❌
(secondary)
- - 0
Improvements ✅
(primary)
- - 0
Improvements ✅
(secondary)
-2.1% [-2.1%, -2.0%] 2
All ❌✅ (primary) - - 0

Cycles

This perf run didn't have relevant results for this metric.

Binary size

This perf run didn't have relevant results for this metric.

Artifact size: 408.59 MiB -> 408.58 MiB (-0.00%)

@JonathanBrouwer

Copy link
Copy Markdown
Member

This is noise, all PRs in the rollup have the same results

@lcnr
lcnr deleted the crashes-migration-cont branch October 8, 2026 20:19
github-actions Bot pushed a commit to rust-lang/rustc-dev-guide that referenced this pull request Oct 10, 2026
…uwer

Rollup of 23 pull requests

Successful merges:

 - rust-lang/rust#163090 (Run LLDB debuginfo tests on `x86_64-pc-windows-msvc` in CI)
 - rust-lang/rust#163806 (even more `tests/crashes` migration for `-Znext-solver`)
 - rust-lang/rust#163508 (Document the `rustc_on_unimplemented` attribute.)
 - rust-lang/rust#163794 (Use `target_family = "wasm"` instead of `target_arch = "wasm32"`)
 - rust-lang/rust#163863 (use the type name instead of `Self` in constructor paths again)
 - rust-lang/rust#163864 (Remove CLAUDE.md)
 - rust-lang/rust#163877 (Add regression test for duplicated rustdoc search results between std and core)
 - rust-lang/rust#163916 (Shrink `PartialRes` and its alignment)
 - rust-lang/rust#149753 (On name resolution error in parameter list, suggest possible `const` typo and avoid unnecessary second error)
 - rust-lang/rust#152557 (Oneshot `is_ready`)
 - rust-lang/rust#157273 (Stabilize `optimize` attribute)
 - rust-lang/rust#162000 (peel_transparent_wrappers only works on non-1ZST)
 - rust-lang/rust#163343 (`unit` and `[1 x unit]` layouts are incompatible on big-endian targets)
 - rust-lang/rust#163348 (perf:cache FFI safety results in improper_ctypes lint)
 - rust-lang/rust#163768 (Move more `rustdoc-html` tests in the right location)
 - rust-lang/rust#163778 (check is_indirect_first_projection when replacing in RefProp)
 - rust-lang/rust#163789 (cg_llvm: Avoid `as_c_char_ptr` in several places)
 - rust-lang/rust#163807 (Add `has_reliable_f16b` for Arm)
 - rust-lang/rust#163809 (Switch TLS implementation for wasi and bump SDK version to 34)
 - rust-lang/rust#163826 (Update books)
 - rust-lang/rust#163849 (ci: update to PowerShell 7.6.6)
 - rust-lang/rust#163869 (INSTALL.md: use UCRT64 instead of MINGW64 in MSYS2 section)
 - rust-lang/rust#163870 (Add if-installed:auto:spellcheck to pre-push script)
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. WG-trait-system-refactor The Rustc Trait System Refactor Initiative (-Znext-solver)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants