Cross-crate inline functions that only call another function.#147655
Cross-crate inline functions that only call another function.#147655cjgillot wants to merge 1 commit intorust-lang:masterfrom
Conversation
|
@bors try @rust-timer queue |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Cross-crate inline functions that only call another function.
|
The job Click to see the possible cause of the failure (guessed by this bot) |
This comment has been minimized.
This comment has been minimized.
|
FYI, I tried this a little while ago and it was bad: #116898 |
|
Finished benchmarking commit (1e4c5f3): comparison URL. Overall result: ❌✅ regressions and improvements - please read the text belowBenchmarking this pull request means it may be perf-sensitive – we'll automatically label it not fit for rolling up. You can override this, but we strongly advise not to, due to possible changes in compiler perf. Next Steps: If you can justify the regressions found in this try perf run, please do so in sufficient writing along with @bors rollup=never 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 8.0%, secondary 9.5%)A less reliable metric. May be of interest, but not used to determine the overall result above.
CyclesResults (primary 68.7%, secondary 31.6%)A less reliable metric. May be of interest, but not used to determine the overall result above.
Binary sizeResults (primary 1.1%, secondary 3.3%)A less reliable metric. May be of interest, but not used to determine the overall result above.
Bootstrap: 475.312s -> 547.264s (15.14%) |
|
Ah thank you, I had forgotten. Your conclusion is still true I guess :) |
r? @ghost