-
Notifications
You must be signed in to change notification settings - Fork 13.4k
Do not run PGO/BOLT in x64 Linux alt builds #141777
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: master
Are you sure you want to change the base?
Conversation
@bors2 try |
Do not run PGO/BOLT in x64 Linux alt builds Should unblock #131077 and also reduce our CI costs. I'll see if I can actually move this job to the free runner, or at least to CodeBuild, if this works. Discussed [here](https://rust-lang.zulipchat.com/#narrow/channel/131828-t-compiler/topic/Utility.20of.20the.20.60dist-x86_64-linux-alt.60.20job/with/521324477). r? `@marcoieni` try-job: `dist-x86_64-linux*`
💔 Test failed
|
e28d0c7
to
318d1e9
Compare
@bors2 try |
Do not run PGO/BOLT in x64 Linux alt builds Should unblock #131077 and also reduce our CI costs. I'll see if I can actually move this job to the free runner, or at least to CodeBuild, if this works. Discussed [here](https://rust-lang.zulipchat.com/#narrow/channel/131828-t-compiler/topic/Utility.20of.20the.20.60dist-x86_64-linux-alt.60.20job/with/521324477). r? `@marcoieni` try-job: `dist-x86_64-linux*`
💔 Test failed
|
318d1e9
to
65bb0d4
Compare
@bors2 try |
Do not run PGO/BOLT in x64 Linux alt builds Should unblock #131077 and also reduce our CI costs. I'll see if I can actually move this job to the free runner, or at least to CodeBuild, if this works. Discussed [here](https://rust-lang.zulipchat.com/#narrow/channel/131828-t-compiler/topic/Utility.20of.20the.20.60dist-x86_64-linux-alt.60.20job/with/521324477). r? `@marcoieni` try-job: `dist-x86_64-linux*`
💔 Test failed
|
Signed-off-by: Jakub Beránek <berykubik@gmail.com>
65bb0d4
to
033ea63
Compare
@bors2 try Fucking submodules. |
Do not run PGO/BOLT in x64 Linux alt builds Should unblock #131077 and also reduce our CI costs. I'll see if I can actually move this job to the free runner, or at least to CodeBuild, if this works. Discussed [here](https://rust-lang.zulipchat.com/#narrow/channel/131828-t-compiler/topic/Utility.20of.20the.20.60dist-x86_64-linux-alt.60.20job/with/521324477). r? `@marcoieni` try-job: `dist-x86_64-linux*`
@bors2 try cancel |
Try build cancelled. |
Nice, the @bors2 try |
Do not run PGO/BOLT in x64 Linux alt builds Should unblock #131077 and also reduce our CI costs. I'll see if I can actually move this job to the free runner, or at least to CodeBuild, if this works. Discussed [here](https://rust-lang.zulipchat.com/#narrow/channel/131828-t-compiler/topic/Utility.20of.20the.20.60dist-x86_64-linux-alt.60.20job/with/521324477). r? `@marcoieni` try-job: `dist-x86_64-linux*`
IIRC results from this build are used for test jobs. So, they will likely slow down a bit but probably not by much. |
💔 Test failed
|
Ok, looks like free runner is a no-go because of disk space, I guess that makes sense.
Thanks, I'll have to check that out. |
6e64f8f
to
eaaa846
Compare
The PGO/BOLT optimizations will be only missing from LLVM, I hope that won't be such an issue, given that most tests execute only very short invocations of @bors2 try |
Do not run PGO/BOLT in x64 Linux alt builds Should unblock #131077 and also reduce our CI costs. I'll see if I can actually move this job to the free runner, or at least to CodeBuild, if this works. Discussed [here](https://rust-lang.zulipchat.com/#narrow/channel/131828-t-compiler/topic/Utility.20of.20the.20.60dist-x86_64-linux-alt.60.20job/with/521324477). r? `@marcoieni` try-job: `dist-x86_64-linux*`
~1h 45m on Codebuild, ideal. |
eaaa846
to
8f22a50
Compare
@bors2 try |
Do not run PGO/BOLT in x64 Linux alt builds Should unblock #131077 and also reduce our CI costs. It seems to run ~1.5h on the x64 larger runner (free runner runs out of disk space, sadly). Discussed [here](https://rust-lang.zulipchat.com/#narrow/channel/131828-t-compiler/topic/Utility.20of.20the.20.60dist-x86_64-linux-alt.60.20job/with/521324477). r? `@marcoieni` try-job: `dist-x86_64-linux*`
this is great, thanks ❤️ |
2h 45m. A bit borderline, but we can always switch to the 8-core runner if it becomes too much. And at least it shouldn't be the bottleneck anymore. @bors r=marcoieni |
… r=marcoieni Do not run PGO/BOLT in x64 Linux alt builds Should unblock rust-lang#131077 and also reduce our CI costs. It seems to run ~1.5h on the x64 larger runner (free runner runs out of disk space, sadly). Discussed [here](https://rust-lang.zulipchat.com/#narrow/channel/131828-t-compiler/topic/Utility.20of.20the.20.60dist-x86_64-linux-alt.60.20job/with/521324477). r? `@marcoieni` try-job: `dist-x86_64-linux*`
Rollup of 8 pull requests Successful merges: - #140638 (UnsafePinned: also include the effects of UnsafeCell) - #141272 (modularize the config module bootstrap) - #141777 (Do not run PGO/BOLT in x64 Linux alt builds) - #141870 (Fix broken link to rustc_type_ir module in serialization docs) - #141938 (update rust offload bootstrap) - #141962 (rustc-dev-guide subtree update) - #141965 (`tests/ui`: A New Order [3/N]) - #141970 (implement new `x` flag: `--skip-std-check-if-no-download-rustc`) r? `@ghost` `@rustbot` modify labels: rollup
… r=marcoieni Do not run PGO/BOLT in x64 Linux alt builds Should unblock rust-lang#131077 and also reduce our CI costs. It seems to run ~1.5h on the x64 larger runner (free runner runs out of disk space, sadly). Discussed [here](https://rust-lang.zulipchat.com/#narrow/channel/131828-t-compiler/topic/Utility.20of.20the.20.60dist-x86_64-linux-alt.60.20job/with/521324477). r? ``@marcoieni`` try-job: `dist-x86_64-linux*`
Rollup of 6 pull requests Successful merges: - #140638 (UnsafePinned: also include the effects of UnsafeCell) - #141777 (Do not run PGO/BOLT in x64 Linux alt builds) - #141938 (update rust offload bootstrap) - #141962 (rustc-dev-guide subtree update) - #141965 (`tests/ui`: A New Order [3/N]) - #141970 (implement new `x` flag: `--skip-std-check-if-no-download-rustc`) r? `@ghost` `@rustbot` modify labels: rollup
Should unblock #131077 and also reduce our CI costs. It seems to run ~1.5h on the x64 larger runner (free runner runs out of disk space, sadly).
Discussed here.
r? @marcoieni
try-job:
dist-x86_64-linux*