-
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
ICE in src/test/ui/issues/issue-74614.rs if MIR inlining is enabled #81788
Comments
I believe the issue is that after inlining, fn test() -> () {
let mut _0: (); // return place in scope 0 at src/test/ui/issues/issue-74614.rs:4:14: 4:14
scope 1 (inlined std::mem::size_of::<T>) { // at src/test/ui/issues/issue-74614.rs:5:5: 5:29
}
bb0: {
_0 = const (); // scope 0 at src/test/ui/issues/issue-74614.rs:4:14: 6:2
return; // scope 0 at src/test/ui/issues/issue-74614.rs:6:2: 6:2
}
} at which point, its @davidtwco does that sound right to you? |
I couldn't say for sure without digging into it myself but that sounds likely. |
This compiles fine on nightly now: https://godbolt.org/z/8nEasbrPT. |
We should just add a regression test and then we can close this. |
This still fails for me locally with current master (d2c24aa):
Not sure why it compiles on Godbolt. Edit: Passes with yesterday's nightly but fails with locally compiled stage 1, will investigate. |
The failed assertion is in line 55 here: rust/compiler/rustc_const_eval/src/interpret/util.rs Lines 53 to 61 in 6f53ddf
Wrapped in if cfg!(debug_assertions) it will not be checked with the prod compiler.
So this unfortunately not fixed and needs more than just a testcase. @rustbot label -E-needs-test |
Fixed by #91743. |
fails with
backtrace
The text was updated successfully, but these errors were encountered: