-
Notifications
You must be signed in to change notification settings - Fork 13.3k
[WIP]Hash borrows in scope for better performance #53306
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
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 |
⌛ Trying commit 5ed7b90 with merge 979c95dac6ec3b5a9968351e021f20f23934e2c1... |
☀️ Test successful - status-travis |
@rust-timer build 979c95dac6ec3b5a9968351e021f20f23934e2c1 |
Success: Queued 979c95dac6ec3b5a9968351e021f20f23934e2c1 with parent 71a1ef1, comparison URL. |
@mikhail-m1 so this seems to be a "wash" performance wise -- no real improvement or loss. The CTFE changes are noise, since those don't even measure NLL. I think the real change that might help here is #53328. I originally thought of this as a step in that direction, but now I'm not sure if it would be helpful or not. Therefore, I'm thinking about closing this PR... what do you think? |
implementation for #53159, but it changes order of checks and affects error messages
r? @nikomatsakis