-
Notifications
You must be signed in to change notification settings - Fork 13.3k
EXPERIMENT: Use rust-cold for reserve_for_push
#115256
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
This is *probably* bad, since it means `reserve_for_push` ends up needing to save all the xmm* registers, but it also reduces register pressure in the code calling `push`, so it'll depend on what the balance is between reallocating pushes and existing-capacity pushes.
@bors try @rust-timer queue |
This comment has been minimized.
This comment has been minimized.
⌛ Trying commit 3e838b3 with merge 0d9872c10177e97b14fa2a422e5bdab1543e21b8... |
☀️ Try build successful - checks-actions |
This comment has been minimized.
This comment has been minimized.
Finished benchmarking commit (0d9872c10177e97b14fa2a422e5bdab1543e21b8): comparison URL. Overall result: ❌✅ regressions and improvements - ACTION NEEDEDBenchmarking 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 may lead to changes in compiler perf. Next Steps: If you can justify the regressions found in this try perf run, please indicate this with @bors rollup=never Instruction countThis is a highly reliable metric that was used to determine the overall result at the top of this comment.
Max RSS (memory usage)ResultsThis is a less reliable metric that may be of interest but was not used to determine the overall result at the top of this comment.
CyclesThis benchmark run did not return any relevant results for this metric. Binary sizeResultsThis is a less reliable metric that may be of interest but was not used to determine the overall result at the top of this comment.
Bootstrap: 631.198s -> 632.2s (0.16%) |
This is probably bad, since it means
reserve_for_push
ends up needing to save all the xmm* registers, but it also reduces register pressure in the code callingpush
, so it'll depend on what the balance is between reallocating pushes and existing-capacity pushes.This cc blocks inlining, but the function is already
inline(never)
.r? @ghost