Skip to content

Add optional CI job to build the compiler with the parallel frontend#153726

Merged
rust-bors[bot] merged 3 commits intorust-lang:mainfrom
Kobzol:parallel-frontend-tests-optional
Mar 12, 2026
Merged

Add optional CI job to build the compiler with the parallel frontend#153726
rust-bors[bot] merged 3 commits intorust-lang:mainfrom
Kobzol:parallel-frontend-tests-optional

Conversation

@Kobzol
Copy link
Member

@Kobzol Kobzol commented Mar 11, 2026

Discussed on Zulip.

Note: this only builds the compiler, stdlib, etc. with 2 threads. UI tests are still compiled serially. I'd add that in a follow-up PR if we see that this new job works well (and if optional auto jobs work well in the first place).

r? @jieyouxu

@rustbot rustbot added A-CI Area: Our Github Actions CI 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 Mar 11, 2026
@Kobzol
Copy link
Member Author

Kobzol commented Mar 11, 2026

@bors try jobs=x86_64-gnu-parallel-frontend

@rust-bors

This comment has been minimized.

rust-bors bot pushed a commit that referenced this pull request Mar 11, 2026
Add optional CI job to run tests with the parallel frontend


try-job: x86_64-gnu-parallel-frontend
@Kobzol Kobzol changed the title Add optional CI job to run tests with the parallel frontend Add optional CI job to build the compiler with the parallel frontend Mar 11, 2026
@rust-bors
Copy link
Contributor

rust-bors bot commented Mar 11, 2026

☀️ Try build successful (CI)
Build commit: 84afa72 (84afa72e5bb52b18560d0f7a0c1668d6842acaa1, parent: 3b1b0ef4d80d3117924d91352c8b6ca528708b3c)

Copy link
Member

@jieyouxu jieyouxu left a comment

Choose a reason for hiding this comment

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

General idea seems good, some clarifications
@rustbot author

View changes since this review

@rustbot rustbot added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Mar 12, 2026
@Kobzol Kobzol force-pushed the parallel-frontend-tests-optional branch from 3a48d69 to 5ac0eed Compare March 12, 2026 08:08
@Kobzol Kobzol force-pushed the parallel-frontend-tests-optional branch from 5ac0eed to 96e4833 Compare March 12, 2026 08:09
@Kobzol
Copy link
Member Author

Kobzol commented Mar 12, 2026

Updated docs and adopted the optional- naming convention.

Copy link
Member

@jieyouxu jieyouxu left a comment

Choose a reason for hiding this comment

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

Thanks, r=me with one doc nit (I forgot to mention)

View changes since this review

@rustbot
Copy link
Collaborator

rustbot commented Mar 12, 2026

The rustc-dev-guide subtree was changed. If this PR only touches the dev guide consider submitting a PR directly to rust-lang/rustc-dev-guide otherwise thank you for updating the dev guide with your changes.

cc @BoxyUwU, @tshepang

@rustbot rustbot added the A-rustc-dev-guide Area: rustc-dev-guide label Mar 12, 2026
@Kobzol
Copy link
Member Author

Kobzol commented Mar 12, 2026

@bors r=jieyouxu

@rust-bors
Copy link
Contributor

rust-bors bot commented Mar 12, 2026

📌 Commit b3333b3 has been approved by jieyouxu

It is now in the queue for this repository.

@rust-bors rust-bors bot 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-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Mar 12, 2026
Zalathar added a commit to Zalathar/rust that referenced this pull request Mar 12, 2026
…ional, r=jieyouxu

Add optional CI job to build the compiler with the parallel frontend

Discussed on [Zulip](https://rust-lang.zulipchat.com/#narrow/channel/187679-t-compiler.2Fparallel-rustc/topic/Add.20the.20parallel.20front-end.20test.20suite/with/578684794).

Note: this only builds the compiler, stdlib, etc. with 2 threads. UI tests are still compiled serially. I'd add that in a follow-up PR if we see that this new job works well (and if optional auto jobs work well in the first place).

r? @jieyouxu
rust-bors bot pushed a commit that referenced this pull request Mar 12, 2026
Rollup of 11 pull requests

Successful merges:

 - #153726 (Add optional CI job to build the compiler with the parallel frontend)
 - #153763 (Don't add empty target features for target-cpu=native on macOS)
 - #153432 (Fix some comments about dataflow analysis.)
 - #153529 (Fix LegacyKeyValueFormat report from docker build: pr)
 - #153694 (fix(query): Pass Query Key to `value_from_cycle_error`)
 - #153708 (de-non_const `Iterator` trait methods)
 - #153717 (unused_macro_rules switched used and unused comments)
 - #153736 (add test that an incomplete feature emits a warning)
 - #153748 (editorconfig: css uses tabs)
 - #153750 (rustc-dev-guide subtree update)
 - #153762 (actually make the is-fn test test what it says it tests)
@rust-bors
Copy link
Contributor

rust-bors bot commented Mar 12, 2026

⌛ Testing commit b3333b3 with merge dc9c2eb...

Workflow: https://github.com/rust-lang/rust/actions/runs/22999418572

rust-bors bot pushed a commit that referenced this pull request Mar 12, 2026
…eyouxu

Add optional CI job to build the compiler with the parallel frontend

Discussed on [Zulip](https://rust-lang.zulipchat.com/#narrow/channel/187679-t-compiler.2Fparallel-rustc/topic/Add.20the.20parallel.20front-end.20test.20suite/with/578684794).

Note: this only builds the compiler, stdlib, etc. with 2 threads. UI tests are still compiled serially. I'd add that in a follow-up PR if we see that this new job works well (and if optional auto jobs work well in the first place).

r? @jieyouxu
Zalathar added a commit to Zalathar/rust that referenced this pull request Mar 12, 2026
…ional, r=jieyouxu

Add optional CI job to build the compiler with the parallel frontend

Discussed on [Zulip](https://rust-lang.zulipchat.com/#narrow/channel/187679-t-compiler.2Fparallel-rustc/topic/Add.20the.20parallel.20front-end.20test.20suite/with/578684794).

Note: this only builds the compiler, stdlib, etc. with 2 threads. UI tests are still compiled serially. I'd add that in a follow-up PR if we see that this new job works well (and if optional auto jobs work well in the first place).

r? @jieyouxu
@JonathanBrouwer
Copy link
Contributor

@bors yield
Yielding to enclosing rollup

@rust-bors
Copy link
Contributor

rust-bors bot commented Mar 12, 2026

Auto build was cancelled. Cancelled workflows:

The next pull request likely to be tested is #153770.

rust-bors bot pushed a commit that referenced this pull request Mar 12, 2026
…uwer

Rollup of 10 pull requests

Successful merges:

 - #153726 (Add optional CI job to build the compiler with the parallel frontend)
 - #153763 (Don't add empty target features for target-cpu=native on macOS)
 - #153432 (Fix some comments about dataflow analysis.)
 - #153529 (Fix LegacyKeyValueFormat report from docker build: pr)
 - #153694 (fix(query): Pass Query Key to `value_from_cycle_error`)
 - #153717 (unused_macro_rules switched used and unused comments)
 - #153736 (add test that an incomplete feature emits a warning)
 - #153748 (editorconfig: css uses tabs)
 - #153750 (rustc-dev-guide subtree update)
 - #153762 (actually make the is-fn test test what it says it tests)
@Zalathar
Copy link
Member

@bors yield (enclosing rollup)

@rust-bors
Copy link
Contributor

rust-bors bot commented Mar 12, 2026

❗ There is currently no auto build in progress on this PR.

@petrochenkov
Copy link
Contributor

if we see that this new job works well

Not sure about the CI environment, but locally I've been working on rustc with RUSTFLAGS="-Zthreads=8" enabled for a couple of years (?) already and have only seen incremental issues (or temporary regressions like #152717), so I'm optimistic.

@rust-bors rust-bors bot merged commit 305778c into rust-lang:main Mar 12, 2026
11 of 12 checks passed
@rustbot rustbot added this to the 1.96.0 milestone Mar 12, 2026
@Kobzol Kobzol deleted the parallel-frontend-tests-optional branch March 12, 2026 15:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-CI Area: Our Github Actions CI A-rustc-dev-guide Area: rustc-dev-guide 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