Skip to content

Add more tests for the parallel frontend#154354

Open
ywxt wants to merge 1 commit intorust-lang:mainfrom
ywxt:more-parallel-tests
Open

Add more tests for the parallel frontend#154354
ywxt wants to merge 1 commit intorust-lang:mainfrom
ywxt:more-parallel-tests

Conversation

@ywxt
Copy link
Contributor

@ywxt ywxt commented Mar 25, 2026

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. labels Mar 25, 2026
@rust-log-analyzer

This comment has been minimized.

@petrochenkov
Copy link
Contributor

And #142949 has flipflopping diagnostics

Could you add that test too, just with //@ ignore-parallel-frontend?
(Unless it fails in single-threaded mode too.)
r=me after that.

@rustbot author

@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 25, 2026
@rustbot
Copy link
Collaborator

rustbot commented Mar 25, 2026

Reminder, once the PR becomes ready for a review, use @rustbot ready.

@ywxt ywxt force-pushed the more-parallel-tests branch from 7712689 to 0f479bf Compare March 26, 2026 06:36
@rustbot
Copy link
Collaborator

rustbot commented Mar 26, 2026

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.

@ywxt
Copy link
Contributor Author

ywxt commented Mar 26, 2026

@rustbot ready

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Mar 26, 2026
@rust-log-analyzer

This comment has been minimized.

@ywxt ywxt force-pushed the more-parallel-tests branch from 0f479bf to 224eda0 Compare March 26, 2026 08:20
@ywxt
Copy link
Contributor Author

ywxt commented Mar 26, 2026

#142949 failed on x86_64-gnu-llvm-21 with the single thread.

the literal 0xffff_ffff_ffff_ffff (decimal 18446744073709551615) does not fit into the type usize and will become 4294967295usize

that's strange on x86_64

@petrochenkov
Copy link
Contributor

@bors r+

@rust-bors
Copy link
Contributor

rust-bors bot commented Mar 26, 2026

📌 Commit 224eda0 has been approved by petrochenkov

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-review Status: Awaiting review from the assignee but also interested parties. labels Mar 26, 2026
JonathanBrouwer added a commit to JonathanBrouwer/rust that referenced this pull request Mar 26, 2026
…henkov

Add more tests for the parallel frontend

Add corresphonding tests from issues:
- rust-lang#115223
- rust-lang#151358
- rust-lang#120757
- rust-lang#134978
- rust-lang#120786
- rust-lang#129911
- rust-lang#129912
- rust-lang#151226
- rust-lang#142064
- rust-lang#120759

For rust-lang#119785
> Effects is no more and the implementation of const_trait_impl has changed dramatically. This is obviously fixed. I don't think it's worth adding a regression test. If you disagree, feel free to open a PR and ping [@]rust-lang/project-const-traits for a "majority vote".

So it doesn't appear here.

And  rust-lang#142949 has flipflopping diagnostics

r? @petrochenkov
JonathanBrouwer added a commit to JonathanBrouwer/rust that referenced this pull request Mar 26, 2026
…henkov

Add more tests for the parallel frontend

Add corresphonding tests from issues:
- rust-lang#115223
- rust-lang#151358
- rust-lang#120757
- rust-lang#134978
- rust-lang#120786
- rust-lang#129911
- rust-lang#129912
- rust-lang#151226
- rust-lang#142064
- rust-lang#120759

For rust-lang#119785
> Effects is no more and the implementation of const_trait_impl has changed dramatically. This is obviously fixed. I don't think it's worth adding a regression test. If you disagree, feel free to open a PR and ping [@]rust-lang/project-const-traits for a "majority vote".

So it doesn't appear here.

And  rust-lang#142949 has flipflopping diagnostics

r? @petrochenkov
rust-bors bot pushed a commit that referenced this pull request Mar 26, 2026
…uwer

Rollup of 5 pull requests

Successful merges:

 - #152757 (Add x86_64-unknown-linux-gnu{m,t}san target which enables {M,T}San by default)
 - #154354 (Add more tests for the parallel frontend)
 - #154088 (Reorder inline asm operands in pretty printer to satisfy grammar constraints)
 - #154407 (Link from `assert_matches` to `debug_assert_matches`)
 - #154420 (Use extended regex syntax)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants