Add regression tests for a number of ICEs and diagnostics issues labelled E-needs-test - #160407
Add regression tests for a number of ICEs and diagnostics issues labelled E-needs-test#160407jakubadamw wants to merge 7 commits into
E-needs-test#160407Conversation
|
r? @nnethercote rustbot has assigned @nnethercote. Use Why was this reviewer chosen?The reviewer was selected based on:
|
| @@ -0,0 +1,20 @@ | |||
| //! Regression test for https://github.com/rust-lang/rust/issues/122214. | |||
There was a problem hiding this comment.
Does this one need to be in tests/incremental? Does it trigger the ICE in the old nightly when it's in tests/ui?
There was a problem hiding this comment.
Good catch! I left it in tests/ui, though, as tests/incremental seems to be used for incremental tests with complex change sequences, which isn’t necessary in this case? There seem to be lots of tests/ui with the incremental flag. I also made it run in both incremental and non-incremental mode – I am not sure if that’s useful? And yes, it ICEs in both.
302d404 to
47c1f8a
Compare
|
This PR changes a file inside |
|
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. |
|
@nnethercote, thank you for your review, I addressed your comments. Crucially, I also went over all the other tests again and discovered that a test case for #129372 still ICEs, with a different set of compilation options. So I turned that into a crash test, instead. @rustbot label -S-waiting-on-author +S-waiting-on-review |
This comment has been minimized.
This comment has been minimized.
…parameter to a trait object
The test was removed on the assumption that it no longer crashed the compiler, but it still does, both on the nightly the issue was reported against and on current master. The assertion has since moved out of the debug info code and into `normalize_erasing_regions`, but it is the same one.
…ype with a `const` parameter
47c1f8a to
28b62fa
Compare
|
The job Click to see the possible cause of the failure (guessed by this bot)Important For more information how to resolve CI failures of this job, visit this link. |
These have since been fixed and need tests.
Closing #111411.
Closing #114665.
Closing #118478.
Closing #122214.
Closing #138891.
Closing #141400.