-
Notifications
You must be signed in to change notification settings - Fork 13k
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
Take a LocalDefId in expect_*item. #91299
Conversation
Some changes occurred in src/tools/clippy. cc @rust-lang/clippy |
(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 d7da795f822454172f02c85dd70ea352aa19e74f with merge def1b42f87128bd490c83a31418c4a5a5463784c... |
This comment has been minimized.
This comment has been minimized.
☀️ Try build successful - checks-actions |
Queued def1b42f87128bd490c83a31418c4a5a5463784c with parent 686e313, future comparison URL. |
d7da795
to
fd30102
Compare
This comment has been minimized.
This comment has been minimized.
Finished benchmarking commit (def1b42f87128bd490c83a31418c4a5a5463784c): comparison url. Summary: This benchmark run did not return any relevant changes. If you disagree with this performance assessment, please file an issue in rust-lang/rustc-perf. Benchmarking this pull request likely means that it is perf-sensitive, so we're automatically marking it as not fit for rolling up. While you can manually mark this PR as fit for rollup, we strongly recommend not doing so since this PR led to changes in compiler perf. @bors rollup=never |
r=me with fulldeps tests updated. |
This comment has been minimized.
This comment has been minimized.
fd30102
to
5fb4648
Compare
@bors r=petrochenkov |
📌 Commit 5fb4648 has been approved by |
⌛ Testing commit 5fb4648 with merge 43a41a1b54de64f057b5220b28dc3cd003f635d2... |
The job Click to see the possible cause of the failure (guessed by this bot)
|
💔 Test failed - checks-actions |
@bors retry |
⌛ Testing commit 5fb4648 with merge 4c618468d2405078fb49ca920f5c401eacd0471c... |
The job Click to see the possible cause of the failure (guessed by this bot)
|
💔 Test failed - checks-actions |
@bors retry |
☀️ Test successful - checks-actions |
Finished benchmarking commit (6db0a0e): comparison url. Summary: This benchmark run did not return any relevant changes. If you disagree with this performance assessment, please file an issue in rust-lang/rustc-perf. @rustbot label: -perf-regression |
Items and item-likes are always HIR owners.
When trying to find such nodes, there is no ambiguity, the
LocalDefId
and theHirId::owner
always match.In such cases,
local_def_id_to_hir_id
does not carry any meaningful information, so we can just skip calling it altogether.