-
Notifications
You must be signed in to change notification settings - Fork 12.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
backtrace missing from file #113881
Comments
Is this from a test case in the suite that's ICEing? I would like to try and reproduce this. |
It looks like this is related to panics that happen during panics: With that file and the listed flags, you will get 3 stacktraces in stderr, but only one in the file:
|
The original finding was from one of the files in https://github.com/matthiaskrgr/glacier2/tree/master/fixed 😅 |
That actually makes sense to me, I'll have to dig into it, but it sounds like it might be straightforward to fix. Edit: wait, re your last comment, did this new machinery regressed a previously fixed ICE, or is it an expected ICE that the new machinery didn't handle gracefully? |
nono, not that I know, the |
This happens exclusively for |
Hm, I also just noticed, when for example clippy panics (you can force that with a delay_span_bug attr), we also generate an empty file :/
repro // this simulates clippy panicking due to ICE
// if we do not catch this in our CI; raise an error
#![feature(rustc_attrs)]
#[rustc_error(delay_span_bug_from_inside_query)]
fn main() {} |
I believe that #114128 fixes the situation for all cases detected, both delayed span bugs and panic during panic. |
…davidtwco When flushing delayed span bugs, write to the ICE dump file even if it doesn't exist Fix rust-lang#113881.
…davidtwco When flushing delayed span bugs, write to the ICE dump file even if it doesn't exist Fix rust-lang#113881.
Sometimes when rustc dumps an ICE backtrace to disk, the file only contains
I have yet to find what actual file/ice causes this, its a bit tricky since there is no mapping from ice file to the actual .rs that was used to cause the ICE.
cc @estebank
The text was updated successfully, but these errors were encountered: