Repository navigation
opt-dist: make llvm builds optional - #143928
Conversation
commented
Jul 14, 2025
|
|
commented
Jul 14, 2025
|
Some changes occurred in src/tools/opt-dist cc @Kobzol |
after finally managing to create a good environment, I tested this and the change works as intended |
47e8d57 to
9295c2f
Compare
9295c2f to
d9c2a27
Compare
commented
Jul 18, 2025
|
@bors2 try |
commented
Jul 18, 2025
adds command line option for disabling llvm builds. it's useful in case of user having their own optimized LLVM, so they won't waste time for (at least) 3 LLVM builds. in this case PGO optimized will be already built in Stage 1, so my previous PR should be addressed for this change
d9c2a27 to
8e840f7
Compare
commented
Jul 18, 2025
commented
Jul 18, 2025
|
@rust-timer build d08b352 |
This comment has been minimized.
This comment has been minimized.
commented
Jul 18, 2025
|
Finished benchmarking commit (d08b352): comparison URL. Overall result: no relevant changes - no action neededBenchmarking 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. @bors rollup=never Instruction countThis benchmark run did not return any relevant results for this metric. Max RSS (memory usage)Results (primary -6.7%, secondary -3.8%)A less reliable metric. May be of interest, but not used to determine the overall result above.
CyclesResults (primary 1.6%, secondary 17.5%)A less reliable metric. May be of interest, but not used to determine the overall result above.
Binary sizeResults (secondary -0.0%)A less reliable metric. May be of interest, but not used to determine the overall result above.
Bootstrap: 464.583s -> 465.606s (0.22%) |
commented
Jul 18, 2025
|
@bors r+ rollup |
adds command line option for disabling llvm builds. it's useful in case of user having their own optimized LLVM, so they won't waste time for (at least) 3 LLVM builds. in this case PGO optimized rustc will be already built in Stage 1, so #143898 should be addressed for this change
couldn't test locally on Linux laptop due to small SSD storage, will try now with windows-msvc host
r? @Kobzol
try-job: dist-x86_64-linux
try-job: dist-x86_64-msvc