Skip to content

regression: MIR assertion failure ICE #94506

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Closed
Mark-Simulacrum opened this issue Mar 1, 2022 · 6 comments
Closed

regression: MIR assertion failure ICE #94506

Mark-Simulacrum opened this issue Mar 1, 2022 · 6 comments
Labels
ICEBreaker-Cleanup-Crew Helping to "clean up" bugs with minimal examples and bisections regression-from-stable-to-beta Performance or correctness regression from stable to beta. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.
Milestone

Comments

@Mark-Simulacrum Mark-Simulacrum added T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. regression-from-stable-to-beta Performance or correctness regression from stable to beta. labels Mar 1, 2022
@Mark-Simulacrum Mark-Simulacrum added this to the 1.60.0 milestone Mar 1, 2022
@rustbot rustbot added the I-prioritize Issue: Indicates that prioritization has been requested for this issue. label Mar 1, 2022
@bjorn3
Copy link
Member

bjorn3 commented Apr 1, 2022

This seems to be the assertion in

/// Returns the source info associated with `location`.
pub fn source_info(&self, location: Location) -> &SourceInfo {
let block = &self[location.block];
let stmts = &block.statements;
let idx = location.statement_index;
if idx < stmts.len() {
&stmts[idx].source_info
} else {
assert_eq!(idx, stmts.len());
&block.terminator().source_info
}
}
In other words the statement_index is higher than the actual statement count. I'm guessing an optimization removes statements but doesn't update the source infos.

@Mark-Simulacrum
Copy link
Member Author

Mark-Simulacrum commented Apr 1, 2022

@rustbot ping bisect

Can we try to get a bisection for this failure? Any of the repositories above should work (e.g., https://github.com/playXE/Jazz)

@rustbot
Copy link
Collaborator

rustbot commented Apr 1, 2022

Hey Cleanup Crew ICE-breakers! This bug has been identified as a good
"Cleanup ICE-breaking candidate". In case it's useful, here are some
instructions for tackling these sorts of bugs. Maybe take a look?
Thanks! <3

cc @AminArria @ayazhafiz @camelid @chrissimpkins @contrun @elshize @h-michael @HallerPatrick @hdhoang @hellow554 @henryboisdequin @imtsuki @JamesPatrickGill @kanru @KarlK90 @matheus-consoli @mental32 @nmccarty @Noah-Kennedy @pard68 @PeytonT @pierreN @Redblueflame @RobbieClarken @RobertoSnap @robjtede @SarthakSingh31 @shekohex @sinato @smmalis37 @steffahn @Stupremee @tamuhey @turboladen @woshilapin @yerke

@rustbot rustbot added the ICEBreaker-Cleanup-Crew Helping to "clean up" bugs with minimal examples and bisections label Apr 1, 2022
@steffahn
Copy link
Member

steffahn commented Apr 1, 2022

Seems to be fixed already

@steffahn
Copy link
Member

steffahn commented Apr 1, 2022

Tested Jazz and slpkg, it’s fixed for both in latest beta; bisections on slpkg:

regression in: 9cdefd7
fixed in: d4de1f2

@Mark-Simulacrum
Copy link
Member Author

Going to go ahead and close then, thanks @steffahn!

@apiraino apiraino removed the I-prioritize Issue: Indicates that prioritization has been requested for this issue. label Apr 6, 2022
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
ICEBreaker-Cleanup-Crew Helping to "clean up" bugs with minimal examples and bisections regression-from-stable-to-beta Performance or correctness regression from stable to beta. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.
Projects
None yet
Development

No branches or pull requests

5 participants