Remove more BuiltinLintDiag variants - part 4#154376
Remove more BuiltinLintDiag variants - part 4#154376GuillaumeGomez wants to merge 2 commits intorust-lang:mainfrom
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.
Remove more BuiltinLintDiag variants - part 4
This comment has been minimized.
This comment has been minimized.
|
Finished benchmarking commit (15fe654): comparison URL. Overall result: ❌ regressions - 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 -2.4%, secondary 2.2%)A less reliable metric. May be of interest, but not used to determine the overall result above.
CyclesResults (primary 2.8%, secondary 5.1%)A less reliable metric. May be of interest, but not used to determine the overall result above.
Binary sizeThis benchmark run did not return any relevant results for this metric. Bootstrap: 482.838s -> 482.192s (-0.13%) |
Part of #153099.
I wonder if there will be any impact on perf by doing this code all the time. Gonna check.
r? @JonathanBrouwer