tests: Remove -Zthreads options from tests in ui/parallel-rustc - #160147
Open
petrochenkov wants to merge 1 commit into
Open
tests: Remove -Zthreads options from tests in ui/parallel-rustc#160147petrochenkov wants to merge 1 commit into
-Zthreads options from tests in ui/parallel-rustc#160147petrochenkov wants to merge 1 commit into
Conversation
petrochenkov
commented
Jul 29, 2026
| .arg("--emit=mir") | ||
| .arg("-Zmir-opt-level=2") | ||
| .arg("-Copt-level=2") | ||
| .arg("-Zthreads=1") |
Contributor
Author
There was a problem hiding this comment.
This was unnecessary because it's already the default.
jieyouxu
approved these changes
Jul 30, 2026
Contributor
jhpratt
added a commit
to jhpratt/rust
that referenced
this pull request
Jul 30, 2026
tests: Remove `-Zthreads` options from tests in `ui/parallel-rustc` The number of threads is now passed by compiletest through `--parallel-frontend-threads=N` in parallel testing runs. If it becomes necessary, we'll be able to set a larger number of threads or iterations for the `ui/parallel-rustc` tests specifically. r? @jieyouxu
JonathanBrouwer
added a commit
to JonathanBrouwer/rust
that referenced
this pull request
Jul 30, 2026
tests: Remove `-Zthreads` options from tests in `ui/parallel-rustc` The number of threads is now passed by compiletest through `--parallel-frontend-threads=N` in parallel testing runs. If it becomes necessary, we'll be able to set a larger number of threads or iterations for the `ui/parallel-rustc` tests specifically. r? @jieyouxu
rust-bors Bot
pushed a commit
that referenced
this pull request
Jul 30, 2026
…uwer Rollup of 8 pull requests Successful merges: - #159817 (Rename splat to avoid stable name collisions) - #158057 (Don't escape U+FF9E and U+FF9F in `escape_debug_ext`) - #159818 (Resolve vars before calling `unnormalized_obligations`) - #160040 (Split function parsing out of `item.rs` to a new module.) - #160044 (Add regression tests for fixed dead-code issues) - #160147 (tests: Remove `-Zthreads` options from tests in `ui/parallel-rustc`) - #160175 (Try to recover less from incorrectly parsed const arg) - #160181 (Mark `Tuple` and `FnPtr` traits `#[fundamental]`)
Member
Contributor
|
This pull request was unapproved. This PR was contained in a rollup (#160207), which was unapproved. |
Member
|
@bors try jobs=x86_64-gnu-parallel-frontend |
This comment has been minimized.
This comment has been minimized.
rust-bors Bot
pushed a commit
that referenced
this pull request
Jul 30, 2026
tests: Remove `-Zthreads` options from tests in `ui/parallel-rustc` try-job: x86_64-gnu-parallel-frontend
Contributor
|
💔 Test for d63d273 failed: CI. Failed job:
|
This comment has been minimized.
This comment has been minimized.
petrochenkov
force-pushed
the
nozthreadsts
branch
from
July 31, 2026 08:51
715f44b to
c019f18
Compare
Collaborator
|
This PR was rebased onto a different main commit. Here's a range-diff highlighting what actually changed. Rebasing is a normal part of keeping PRs up to date, so no action is needed—this note is just to help reviewers. |
Contributor
Author
|
Disabled the failing test in parallel mode. |
Contributor
JonathanBrouwer
added a commit
to JonathanBrouwer/rust
that referenced
this pull request
Jul 31, 2026
tests: Remove `-Zthreads` options from tests in `ui/parallel-rustc` The number of threads is now passed by compiletest through `--parallel-frontend-threads=N` in parallel testing runs. If it becomes necessary, we'll be able to set a larger number of threads or iterations for the `ui/parallel-rustc` tests specifically. r? @jieyouxu
JonathanBrouwer
added a commit
to JonathanBrouwer/rust
that referenced
this pull request
Jul 31, 2026
tests: Remove `-Zthreads` options from tests in `ui/parallel-rustc` The number of threads is now passed by compiletest through `--parallel-frontend-threads=N` in parallel testing runs. If it becomes necessary, we'll be able to set a larger number of threads or iterations for the `ui/parallel-rustc` tests specifically. r? @jieyouxu
JonathanBrouwer
added a commit
to JonathanBrouwer/rust
that referenced
this pull request
Jul 31, 2026
tests: Remove `-Zthreads` options from tests in `ui/parallel-rustc` The number of threads is now passed by compiletest through `--parallel-frontend-threads=N` in parallel testing runs. If it becomes necessary, we'll be able to set a larger number of threads or iterations for the `ui/parallel-rustc` tests specifically. r? @jieyouxu
rust-bors Bot
pushed a commit
that referenced
this pull request
Jul 31, 2026
…uwer Rollup of 21 pull requests Successful merges: - #160100 (Add "system" option to `override-allocator` directive) - #159675 (rustc: Support `--jobs` options for limiting parallelism in various parts of the compiler) - #159999 (Fix invalidation of stdlib in bootstrap when using non-LLVM codegen backends) - #160233 (Bubble bad path error while parsing field to avoid unecessary second error) - #160272 (rustc_metadata: Move native library search code to `rustc_codegen_ssa`) - #154202 (rustfmt: Format `cfg_select!`) - #158835 (rustc_passes: lint unused `#[path]` attributes on inline modules) - #159520 (Suggest `Vec<T>` instead of `[T]`) - #160034 (Move "macro only" check for `#[allow_internal_unsafe/unstable]` to attribute parser) - #160066 (rustc_middle: lint attribute cleanups) - #160085 (Remove various superfluous lint attributes) - #160113 (Coalesce `rustc_on_unimplemented` attributes and lint malformed filters) - #160119 (fix query cycle in `coroutine_hidden_types` for the next solver) - #160147 (tests: Remove `-Zthreads` options from tests in `ui/parallel-rustc`) - #160157 (Remove outdated comments from `va_list.rs`) - #160159 (More accurately check for interior mutability in `invalid_reference_casting` lint) - #160208 (rustdoc: Fix crash when trying to list attributes on an opaque type) - #160244 (Rename splat to rustc_splat in error messages) - #160246 (dont fire `unused_mut` on `&pin mut self`) - #160247 (Configure backport nominations for rustfmt) - #160274 (renovate: update lock files weekly)
JonathanBrouwer
added a commit
to JonathanBrouwer/rust
that referenced
this pull request
Jul 31, 2026
tests: Remove `-Zthreads` options from tests in `ui/parallel-rustc` The number of threads is now passed by compiletest through `--parallel-frontend-threads=N` in parallel testing runs. If it becomes necessary, we'll be able to set a larger number of threads or iterations for the `ui/parallel-rustc` tests specifically. r? @jieyouxu
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The number of threads is now passed by compiletest through
--parallel-frontend-threads=Nin parallel testing runs.If it becomes necessary, we'll be able to set a larger number of threads or iterations for the
ui/parallel-rustctests specifically.r? @jieyouxu