Move check only relevant in error case out of critical path#82738
Move check only relevant in error case out of critical path#82738bors merged 1 commit intorust-lang:masterfrom
Conversation
Move the check for potentially forgotten `return` in a tail expression of arbitrary expressions into the coercion error branch to avoid computing unncessary coercion checks on successful code. Follow up to rust-lang#81458.
|
r? @davidtwco (rust-highfive has picked a reviewer for you, use r? to override) |
|
@bors try @rust-timer queue |
|
Awaiting bors try build completion. @rustbot label: +S-waiting-on-perf |
|
⌛ Trying commit c6c243a with merge 75dfc563af3058ba1fd662a6c5c9e256e96338af... |
|
☀️ Try build successful - checks-actions |
|
Queued 75dfc563af3058ba1fd662a6c5c9e256e96338af with parent 1c77a1f, future comparison URL. |
|
Finished benchmarking try commit (75dfc563af3058ba1fd662a6c5c9e256e96338af): comparison url. Benchmarking this pull request likely means that it is perf-sensitive, so we're automatically marking it as not fit for rolling up. Please note that if the perf results are neutral, you should likely undo the rollup=never given below by specifying Importantly, though, if the results of this run are non-neutral do not roll this PR up -- it will mask other regressions or improvements in the roll up. @bors rollup=never |
|
📌 Commit c6c243a has been approved by |
|
⌛ Testing commit c6c243a with merge 36aedfbed8fbd3df10ae768cd2fb7b8358420ed4... |
|
💥 Test timed out |
|
@bors retry |
|
☀️ Test successful - checks-actions |
Move the check for potentially forgotten
returnin a tail expressionof arbitrary expressions into the coercion error branch to avoid
computing unncessary coercion checks on successful code.
Follow up to #81458.