Optimize Cranelift with PGO - #163649
Optimize Cranelift with PGO#163649
Conversation
|
@bors try jobs=dist-x86_64-linux @rust-timer queue backends=Llvm,Cranelift |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Optimize Cranelift with PGO try-job: dist-x86_64-linux
This comment has been minimized.
This comment has been minimized.
|
Finished benchmarking commit (65872f1): comparison URL. Overall result: ❌✅ regressions and improvements - please read:Benchmarking 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. Next, please: If you can, justify the regressions found in this try perf run in writing along with @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.6%, secondary -0.3%)A less reliable metric. May be of interest, but not used to determine the overall result above.
CyclesResults (primary -1.2%, secondary -0.1%)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: 490.969s -> 492.497s (0.31%) |
|
No changes on LLVM and nice wins for Cranelift. CI duration is extended by maybe 2-3 minutes, only for full builds, so that seems fine. |
|
This PR modifies If appropriate, please update This PR modifies If appropriate, please update |
|
rustbot has assigned @Mark-Simulacrum. Use Why was this reviewer chosen?The reviewer was selected based on:
|
|
@bors r+ |
This comment has been minimized.
This comment has been minimized.
What is this?This is an experimental post-merge analysis report that shows differences in test outcomes between the merged PR and its parent PR.Comparing 1d9e680 (parent) -> 56343b1 (this PR) Test differencesShow 1 test diffStage 2
Job group index Test dashboardRun cargo run --manifest-path src/ci/citool/Cargo.toml -- \
test-dashboard 56343b1a7d3fed3f349a5c2dc474f65b504f5221 --output-dir test-dashboardAnd then open Job duration changes
How to interpret the job duration changes?Job durations can vary a lot, based on the actual runner instance |
|
Finished benchmarking commit (56343b1): comparison URL. Overall result: no relevant changes - no action needed@rustbot label: -perf-regression Instruction countThis perf run didn't have relevant results for this metric. Max RSS (memory usage)Results (primary -3.6%)A less reliable metric. May be of interest, but not used to determine the overall result above.
CyclesResults (primary 5.2%, secondary 2.7%)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: 488.976s -> 488.752s (-0.05%) |
Opening for perf. experiments.