Feat/inline always+target feature #2 no callsite - #162896
Jamesbarford wants to merge 3 commits into
Conversation
|
@bors try @rust-timer queue |
This comment has been minimized.
This comment has been minimized.
…e-#2_no-callsite, r=<try> Feat/inline always+target feature #2 no callsite
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
27e7311 to
9dea6d6
Compare
This comment has been minimized.
This comment has been minimized.
|
Finished benchmarking commit (2a0e52c): comparison URL. Overall result: ✅ improvements - no action neededBenchmarking means the PR may be perf-sensitive. It's automatically marked not fit for rolling up. Overriding is possible but disadvised: it risks changing compiler perf. @bors rollup=never rustc-perf Instruction countOur most reliable metric. Used to determine the overall result above. However, even this metric can be noisy.
Max RSS (memory usage)Results (primary 0.2%, secondary -1.3%)A less reliable metric. May be of interest, but not used to determine the overall result above.
CyclesResults (secondary 0.5%)A less reliable metric. May be of interest, but not used to determine the overall result above.
Binary sizeThis perf run didn't have relevant results for this metric. Bootstrap: 500.132s -> 499.359s (-0.15%) |
I've branched off my actually branch for inline_always + target_features to see what removing a rather odd application of adding inline attributes to the callsite and definition impact on perf is
r? @ghost