Skip to content

opt-dist: make llvm builds optional - #143928

Merged
bors merged 1 commit into
rust-lang:masterfrom
ognevny:opt-dist-build-llvm
Jul 19, 2025
Merged

bors merged 1 commit into
rust-lang:masterfrom
ognevny:opt-dist-build-llvm

Conversation

@ognevny

@ognevny ognevny commented Jul 14, 2025 •

Copy link
Copy Markdown
Contributor

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

@rustbot

ghost commented Jul 14, 2025

Copy link
Copy Markdown
Collaborator

Kobzol is not on the review rotation at the moment.
They may take a while to respond.

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jul 14, 2025
@rustbot

ghost commented Jul 14, 2025

Copy link
Copy Markdown
Collaborator

Some changes occurred in src/tools/opt-dist

cc @Kobzol

@ognevny

ghost commented Jul 14, 2025 •

Copy link
Copy Markdown
Contributor Author

couldn't test locally on Linux laptop due to small SSD storage, will try now with windows-msvc host

after finally managing to create a good environment, I tested this and the change works as intended

Comment thread src/tools/opt-dist/src/exec.rs
@ognevny
ognevny force-pushed the opt-dist-build-llvm branch from 47e8d57 to 9295c2f Compare July 17, 2025 20:58
Comment thread src/tools/opt-dist/src/main.rs Outdated
@ognevny
ognevny force-pushed the opt-dist-build-llvm branch from 9295c2f to d9c2a27 Compare July 18, 2025 08:01

ghost left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Left two nits, will start a try run in the meantime.

Comment thread src/tools/opt-dist/src/main.rs Outdated
Comment thread src/tools/opt-dist/src/main.rs Outdated
@Kobzol

ghost commented Jul 18, 2025

Copy link
Copy Markdown
Member

@bors2 try

@rust-bors

ghost commented Jul 18, 2025

Copy link
Copy Markdown
Contributor

⌛ Trying commit d9c2a27 with merge d08b352…

To cancel the try build, run the command @bors2 try cancel.

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
@ognevny
ognevny force-pushed the opt-dist-build-llvm branch from d9c2a27 to 8e840f7 Compare July 18, 2025 08:20
@rust-bors

ghost commented Jul 18, 2025

Copy link
Copy Markdown
Contributor

☀️ Try build successful (CI)
Build commit: d08b352 (d08b35280fb8e6cb4ea877781c9531e0e69b9278, parent: 1aa5b2246560ce85b42fa8b33e5927c5de3fa389)

@Kobzol

ghost commented Jul 18, 2025

Copy link
Copy Markdown
Member

@rust-timer build d08b352

@rust-timer

This comment has been minimized.

@rust-timer

ghost commented Jul 18, 2025

Copy link
Copy Markdown
Collaborator

Finished benchmarking commit (d08b352): comparison URL.

Overall result: no relevant changes - no action needed

Benchmarking 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
@rustbot label: -S-waiting-on-perf -perf-regression

Instruction count

This 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.

mean range count
Regressions ❌
(primary)
- - 0
Regressions ❌
(secondary)
- - 0
Improvements ✅
(primary)
-6.7% [-6.7%, -6.7%] 1
Improvements ✅
(secondary)
-3.8% [-3.8%, -3.8%] 1
All ❌✅ (primary) -6.7% [-6.7%, -6.7%] 1

Cycles

Results (primary 1.6%, secondary 17.5%)

A less reliable metric. May be of interest, but not used to determine the overall result above.

mean range count
Regressions ❌
(primary)
1.6% [1.6%, 1.6%] 1
Regressions ❌
(secondary)
17.5% [2.2%, 26.6%] 9
Improvements ✅
(primary)
- - 0
Improvements ✅
(secondary)
- - 0
All ❌✅ (primary) 1.6% [1.6%, 1.6%] 1

Binary size

Results (secondary -0.0%)

A less reliable metric. May be of interest, but not used to determine the overall result above.

mean range count
Regressions ❌
(primary)
- - 0
Regressions ❌
(secondary)
- - 0
Improvements ✅
(primary)
- - 0
Improvements ✅
(secondary)
-0.0% [-0.0%, -0.0%] 1
All ❌✅ (primary) - - 0

Bootstrap: 464.583s -> 465.606s (0.22%)
Artifact size: 374.66 MiB -> 374.71 MiB (0.02%)

@Kobzol

ghost commented Jul 18, 2025

Copy link
Copy Markdown
Member

@bors r+ rollup

@bors

ghost commented Jul 18, 2025

Copy link
Copy Markdown
Collaborator

📌 Commit 8e840f7 has been approved by Kobzol

It is now in the queue for this repository.

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Jul 18, 2025
@bors
bors merged commit 232bef3 into rust-lang:master Jul 19, 2025
@rustbot rustbot added this to the 1.90.0 milestone Jul 19, 2025
@ognevny
ognevny deleted the opt-dist-build-llvm branch July 19, 2025 08:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Couldn't load reviewers.

Assignees

Couldn't load assignees.