-
Notifications
You must be signed in to change notification settings - Fork 12.7k
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
[PERF] further drop changes #66703
[PERF] further drop changes #66703
Conversation
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 try @rust-timer queue |
Awaiting bors try build completion |
[PERF] further drop changes r? @ghost
☀️ Try build successful - checks-azure |
Queued a707b15 with parent b56b239, future comparison URL. |
Finished benchmarking try commit a707b15, comparison URL. |
6002ebe
to
ce04564
Compare
This comment has been minimized.
This comment has been minimized.
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 try @rust-timer queue |
Awaiting bors try build completion |
⌛ Trying commit 0cdbe985ed9bab2bbd8658cb4398d1323b41b290 with merge b7050c98084b69b4ad610de6d40401c55e701c84... |
☀️ Try build successful - checks-azure |
Queued b7050c98084b69b4ad610de6d40401c55e701c84 with parent d8bdb3f, future comparison URL. |
Finished benchmarking try commit b7050c98084b69b4ad610de6d40401c55e701c84, comparison URL. |
@bors try @rust-timer queue |
Awaiting bors try build completion |
⌛ Trying commit 9bb736a7b4c8ebf1985c0fab4befded046992aaa with merge 995015e745a28d4b94e63e2f7daf827dbff40f3d... |
src/librustc/ty/instance.rs
Outdated
.map_or(ty.is_enum(), |dtor| tcx.codegen_fn_attrs(dtor.did).requests_inline()); | ||
.map_or(true, |adt_def| { | ||
adt_def.destructor(tcx) | ||
.map_or( |
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.
Maybe using lazy map_or_else
is better.
src/librustc_mir/borrow_check/mod.rs
Outdated
flag: _, | ||
target: _, | ||
unwind: _, |
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.
Maybe using ..
in this and other places is better.
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 |
☀️ Try build successful - checks-azure |
Queued 995015e745a28d4b94e63e2f7daf827dbff40f3d with parent 4007d4e, future comparison URL. |
Finished benchmarking try commit 995015e745a28d4b94e63e2f7daf827dbff40f3d, comparison URL. |
9bb736a
to
9a926f9
Compare
@bors try @rust-timer queue |
Awaiting bors try build completion |
⌛ Trying commit ba989b2 with merge e66cbac2cde26b586f2989d64da447e77bac413b... |
☀️ Try build successful - checks-azure |
Queued e66cbac2cde26b586f2989d64da447e77bac413b with parent e9bca51, future comparison URL. |
@matthewjasper any updates? |
Not really, closing for now. |
r? @ghost