Skip to content

Build rustc with assertions in dist-alt jobs #142235

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

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Kobzol
Copy link
Contributor

@Kobzol Kobzol commented Jun 9, 2025

Revival of #131077, to check CI times now that we don't do PGO/BOLT anymore on Linux -alt builds.

r? @ghost

try-job: dist-x86_64-msvc-alt
try-job: dist-x86_64-linux-alt

@rustbot rustbot added A-testsuite Area: The testsuite used to check the correctness of rustc S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-infra Relevant to the infrastructure team, which will review and decide on the PR/issue. labels Jun 9, 2025
@rustbot
Copy link
Collaborator

rustbot commented Jun 9, 2025

Some changes occurred in src/tools/opt-dist

cc @Kobzol

@Kobzol
Copy link
Contributor Author

Kobzol commented Jun 9, 2025

@bors2 try

@rust-bors
Copy link

rust-bors bot commented Jun 9, 2025

⌛ Trying commit fea114c with merge 719c0f0

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

rust-bors bot added a commit that referenced this pull request Jun 9, 2025
Rustc dist alt assertions

Revival of #131077, to check CI times now that we don't do PGO/BOLT anymore on Linux `-alt` builds.

r? `@ghost`

try-job: dist-x86_64-msvc-alt
try-job: dist-x86_64-linux-alt
@Kobzol Kobzol changed the title Rustc dist alt assertions Build rustc with assertions in dist-alt jobs Jun 9, 2025
@rust-bors
Copy link

rust-bors bot commented Jun 9, 2025

☀️ Try build successful (CI)
Build commit: 719c0f0 (719c0f0e93c1f3606edfa5381222b9d9b8b0c087)

@Kobzol
Copy link
Contributor Author

Kobzol commented Jun 9, 2025

@bors2 try

@rust-bors
Copy link

rust-bors bot commented Jun 9, 2025

⌛ Trying commit 6d4df2d with merge 6acdc26

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

rust-bors bot added a commit that referenced this pull request Jun 9, 2025
Build rustc with assertions in `dist-alt` jobs

Revival of #131077, to check CI times now that we don't do PGO/BOLT anymore on Linux `-alt` builds.

r? `@ghost`

try-job: dist-x86_64-msvc-alt
try-job: dist-x86_64-linux-alt
@rust-bors
Copy link

rust-bors bot commented Jun 9, 2025

☀️ Try build successful (CI)
Build commit: 6acdc26 (6acdc261c9a2a67217d87dea62949057a09f503f)

@Kobzol
Copy link
Contributor Author

Kobzol commented Jun 9, 2025

2h 14 minutes with the 8-core runner, pretty nice. @marcoieni would you be fine to switch the dist-alt runner to 8 cores in order to get rustc assertions in CI builds?

@marcoieni
Copy link
Member

2h 14 minutes with the 8-core runner, pretty nice. @marcoieni would you be fine to switch the dist-alt runner to 8 cores in order to get rustc assertions in CI builds?

yes it's fine 👍

lqd and others added 2 commits June 10, 2025 12:17
llvm assertions are already enabled and debug assertions are also useful
@Kobzol Kobzol force-pushed the rustc-dist-alt-assertions branch from 6d4df2d to 607a567 Compare June 10, 2025 10:17
@Kobzol
Copy link
Contributor Author

Kobzol commented Jun 10, 2025

@bors2 try

rust-bors bot added a commit that referenced this pull request Jun 10, 2025
Build rustc with assertions in `dist-alt` jobs

Revival of #131077, to check CI times now that we don't do PGO/BOLT anymore on Linux `-alt` builds.

r? `@ghost`

try-job: dist-x86_64-msvc-alt
try-job: dist-x86_64-linux-alt
@rust-bors
Copy link

rust-bors bot commented Jun 10, 2025

⌛ Trying commit 607a567 with merge 11c9e81

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

@rust-bors
Copy link

rust-bors bot commented Jun 10, 2025

☀️ Try build successful (CI)
Build commit: 11c9e81 (11c9e81093a637885ff88d565c318d0b9c074c63)

@Kobzol
Copy link
Contributor Author

Kobzol commented Jun 10, 2025

r? @marcoieni

@marcoieni
Copy link
Member

@bors r+

@bors
Copy link
Collaborator

bors commented Jun 10, 2025

📌 Commit 607a567 has been approved by marcoieni

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 Jun 10, 2025
fmease added a commit to fmease/rust that referenced this pull request Jun 10, 2025
…r=marcoieni

Build rustc with assertions in `dist-alt` jobs

Revival of rust-lang#131077, to check CI times now that we don't do PGO/BOLT anymore on Linux `-alt` builds.

r? `@ghost`

try-job: dist-x86_64-msvc-alt
try-job: dist-x86_64-linux-alt
bors added a commit that referenced this pull request Jun 11, 2025
Rollup of 13 pull requests

Successful merges:

 - #128425 (Make `missing_fragment_specifier` an unconditional error)
 - #141639 (Expose discriminant values in stable_mir)
 - #141967 (Configure bootstrap backport nominations through triagebot)
 - #142042 (Make E0621 missing lifetime suggestion verbose)
 - #142176 (tests: Split dont-shuffle-bswaps along opt-levels and arches)
 - #142235 (Build rustc with assertions in `dist-alt` jobs)
 - #142248 (Add supported asm types for LoongArch32)
 - #142272 (tests: Change ABIs in tests to more future-resilient ones)
 - #142282 (Only run `citool` tests on the `auto` branch)
 - #142285 (tests: Do not run afoul of asm.validity.non-exhaustive in input-stats)
 - #142297 (Implement `//@ needs-target-std` compiletest directive)
 - #142298 (Make loongarch-none target maintainers more easily pingable)
 - #142306 (Dont unwrap and re-wrap typing envs)

Failed merges:

 - #141942 (Implement representation options to smir)

r? `@ghost`
`@rustbot` modify labels: rollup
@workingjubilee
Copy link
Member

sorry, I don't have a good reason, it just has a vibe, and I think everyone else is assuming it does too.

@bors rollup=iffy

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-testsuite Area: The testsuite used to check the correctness of rustc S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-infra Relevant to the infrastructure team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants