-
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
Use less string interning #98638
Use less string interning #98638
Conversation
Interning here doesn't save us anything. It rather costs a bit of time.
Some changes occurred in compiler/rustc_codegen_gcc cc @antoyo |
r? @jackh726 (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 566bb8d5a574df9b6814ee85a4bc22c61b85ef5d with merge 50ddf6d97cca693466234951f314e347574e0255... |
☀️ Try build successful - checks-actions |
Queued 50ddf6d97cca693466234951f314e347574e0255 with parent 94e9374, future comparison URL. |
Finished benchmarking commit (50ddf6d97cca693466234951f314e347574e0255): comparison url. Instruction countThis benchmark run did not return any relevant results for this metric. Max RSS (memory usage)Results
CyclesResults
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 may lead to changes in compiler perf. @bors rollup=never Footnotes |
Thanks, r=me with or without addressing the nit. |
566bb8d
to
f688f4d
Compare
@bors r=tmiasko |
📌 Commit f688f4d has been approved by |
⌛ Testing commit f688f4d with merge 2808ff66b08f8095a220aa2c7c733857803be5b2... |
The job Click to see the possible cause of the failure (guessed by this bot)
|
💔 Test failed - checks-actions |
☀️ Test successful - checks-actions |
Finished benchmarking commit (1dcff2d): comparison url. Instruction count
Max RSS (memory usage)Results
CyclesResults
If you disagree with this performance assessment, please file an issue in rust-lang/rustc-perf. @rustbot label: -perf-regression Footnotes |
Use less string interning This removes string interning in a couple of places where doing so won't result in perf improvements. I also switched one place to use pre-interned symbols.
This removes string interning in a couple of places where doing so won't result in perf improvements. I also switched one place to use pre-interned symbols.