-
Notifications
You must be signed in to change notification settings - Fork 13.5k
Make drop_flags an IndexVec. #110962
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
Make drop_flags an IndexVec. #110962
Conversation
r? @b-naber (rustbot has picked a reviewer for you, use r? to override) |
Some changes occurred to MIR optimizations cc @rust-lang/wg-mir-opt |
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.
r=me with ci green
@@ -22,9 +22,9 @@ | |||
} | |||
|
|||
bb0: { | |||
+ _9 = const false; // scope 0 at $DIR/issue_41888.rs:+1:9: +1:10 | |||
+ _7 = const false; // scope 0 at $DIR/issue_41888.rs:+1:9: +1:10 |
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.
nice 😄 this is good proof the order is more stable now, hehe
@bors r+ |
…-errors Make drop_flags an IndexVec. Fixes rust-lang#91943
Rollup of 6 pull requests Successful merges: - rust-lang#110614 (Clear response values for overflow in new solver) - rust-lang#110894 (Bump libffi-sys to 2.3.0) - rust-lang#110932 (include source error for LoadLibraryExW) - rust-lang#110958 (Make sure that some stdlib method signatures aren't accidental refinements) - rust-lang#110962 (Make drop_flags an IndexVec.) - rust-lang#110965 (Don't duplicate anonymous lifetimes for async fn in traits) Failed merges: r? `@ghost` `@rustbot` modify labels: rollup
Fixes #91943