-
Notifications
You must be signed in to change notification settings - Fork 12.8k
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
Cleanup from lexical MIR borrowck removal #55959
Conversation
0f1ca14
to
efd126f
Compare
The last commits sound potentially perf-sensitive, so @bors try |
⌛ Trying commit efd126fc322ae247d50c747d181a0d507b9641d6 with merge 5247df0b727520b9128fbf5b55797a25859a3dd8... |
☀️ Test successful - status-travis |
@rust-timer build 5247df0b727520b9128fbf5b55797a25859a3dd8 |
Success: Queued 5247df0b727520b9128fbf5b55797a25859a3dd8 with parent 6f93e93, comparison URL. |
Finished benchmarking try commit 5247df0b727520b9128fbf5b55797a25859a3dd8 |
Looks pretty green to me except serde-check which is +1.4% |
dddf04d
to
1335c18
Compare
Added a somewhat speculative performance fix, but it looks OK locally. |
⌛ Trying commit 1335c18e48186480d136f99d2b13cbc1a55a66eb with merge 9090986ee400624e4cf0a86cb63e879468f473dd... |
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
1335c18
to
4f0b256
Compare
@bors try |
Cleanup from lexical MIR borrowck removal Lexical MIR borrowck was removed months ago now, and `EndRegion`s are no longer used for MIRI verification. * Remove `rustc::mir::StatementKind::EndRegion` and the `-Zemit_end_regions` flag * Use `RegionVid` instead of `Region` in BorrowSet * Rewrite drop generation to create fewer goto terminators. r? @nikomatsakis
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
☀️ Test successful - status-travis |
@rust-timer build 6f6b42d |
Insufficient permissions to issue commands to rust-timer. |
@rust-timer build 6f6b42d |
Success: Queued 6f6b42d with parent 6b9b97b, comparison URL. |
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 but tests are failing for some reason?
mir::TerminatorKind::Return | | ||
mir::TerminatorKind::GeneratorDrop => { | ||
// When we return from the function, then all `ReScope`-style regions | ||
// are guaranteed to have ended. |
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, lots of dead code
Looks like the performance issue was elsewhere as well. |
4f0b256
to
1bc2b8d
Compare
Since lexical MIR borrow check is gone, and validation no longer uses these, they can be removed.
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
Now that EndRegion is gone, we don't need to create as many gotos.
c4bdd08
to
1d7fc0c
Compare
@bors try |
⌛ Trying commit 1d7fc0c with merge af7b94fb078ef359e0327913df90f90d266a8141... |
☀️ Test successful - status-travis |
@rust-timer build af7b94fb078ef359e0327913df90f90d266a8141 |
Success: Queued af7b94fb078ef359e0327913df90f90d266a8141 with parent f37247f, comparison URL. |
perf is green now |
@bors r=nikomatsakis |
📌 Commit 1d7fc0c has been approved by |
Cleanup from lexical MIR borrowck removal Lexical MIR borrowck was removed months ago now, and `EndRegion`s are no longer used for MIRI verification. * Remove `rustc::mir::StatementKind::EndRegion` and the `-Zemit_end_regions` flag * Use `RegionVid` instead of `Region` in BorrowSet * Rewrite drop generation to create fewer goto terminators. r? @nikomatsakis
💔 Test failed - status-travis |
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
@bors retry |
Cleanup from lexical MIR borrowck removal Lexical MIR borrowck was removed months ago now, and `EndRegion`s are no longer used for MIRI verification. * Remove `rustc::mir::StatementKind::EndRegion` and the `-Zemit_end_regions` flag * Use `RegionVid` instead of `Region` in BorrowSet * Rewrite drop generation to create fewer goto terminators. r? @nikomatsakis
☀️ Test successful - status-appveyor, status-travis |
Lexical MIR borrowck was removed months ago now, and
EndRegion
s are no longer used for MIRI verification.rustc::mir::StatementKind::EndRegion
and the-Zemit_end_regions
flagRegionVid
instead ofRegion
in BorrowSetr? @nikomatsakis