-
Notifications
You must be signed in to change notification settings - Fork 14k
Offload bootstrap #148671
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Offload bootstrap #148671
Conversation
|
This PR changes how LLVM is built. Consider updating src/bootstrap/download-ci-llvm-stamp. |
|
@bors2 try @rust-timer queue (@Kobzol feel free to change if you want to see something else). |
|
Awaiting bors try build completion. @rustbot label: +S-waiting-on-perf |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
|
💔 Test for 7069335 failed: CI. Failed jobs:
|
This comment has been minimized.
This comment has been minimized.
|
@bors2 try @rust-timer queue |
|
Awaiting bors try build completion. @rustbot label: +S-waiting-on-perf |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
|
💔 Test for 31000c9 failed: CI. Failed jobs:
|
|
@bors2 try @rust-timer queue |
|
Awaiting bors try build completion. @rustbot label: +S-waiting-on-perf |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
|
💔 Test for 57b6a7b failed: CI. Failed jobs:
|
|
@bors2 try @rust-timer queue |
|
Awaiting bors try build completion. @rustbot label: +S-waiting-on-perf |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
|
💔 Test for 3b40cbb failed: CI. Failed jobs:
|
|
@bors2 try @rust-timer queue |
|
Awaiting bors try build completion. @rustbot label: +S-waiting-on-perf |
This comment has been minimized.
This comment has been minimized.
|
💔 Test for 67eb183 failed: CI. Failed jobs:
|
This comment has been minimized.
This comment has been minimized.
|
@bors2 try @rust-timer queue |
|
Awaiting bors try build completion. @rustbot label: +S-waiting-on-perf |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
|
💔 Test for cf671bb failed: CI. Failed jobs:
|
|
@bors2 try @rust-timer queue |
|
Awaiting bors try build completion. @rustbot label: +S-waiting-on-perf |
This comment has been minimized.
This comment has been minimized.
|
The job Click to see the possible cause of the failure (guessed by this bot) |
|
💔 Test for f795c48 failed: CI. Failed jobs:
|
|
@Kobzol You mentioned before that we build llvm here like 5 times, and I'm indeed getting a bit confused about whether I am changing the right things. The way I see it from the logs (e.g. #148671 (comment)) is
I see that despite enabling lld in bootstrap.toml, we don't build the lld project anymore. Is that intended here, or am I misinterpreting the logs? As you can see in the commit history, I tried to explicitly enable lld as project in one commit. I know bootstrap is a bit tricky, but can you give me a vague explanation of what's the goal of these steps here, so I can guess better what to change? |
The recommended way to build the offload project changed.
While already at it, testing how much overhead we get from building (or shipping) the two runtimes.
r? @Kobzol