-
Notifications
You must be signed in to change notification settings - Fork 17
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
Mad rustc update #272
Mad rustc update #272
Conversation
3aedfd8
to
f11f64d
Compare
7473e15
to
38f1141
Compare
Otherwise they don't have the right nightly.
083eeca
to
326406c
Compare
I also had to update cargo dependencies because of a nightly feature change that broke `ahash`.
326406c
to
0fb1b5d
Compare
@0 := const (true) | ||
}, | ||
0 => { | ||
@0 := const (false) | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm surprised that the order changed here (case 1 then case 0)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it changed in the MIR (and this is likely my doing again).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh wait, that's code that we generate ourselves. We generate it in the order of the variants in the type definition, maybe that changed? That is weird.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I opened an issue #278
Companion PR to hacspec/hax#735. This updates the rustc pin by about 8 months.
Closes #182.