Skip to content
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

Add FileCheck annotations to mir-opt/issues #134943

Merged
merged 1 commit into from
Mar 1, 2025

Conversation

Shunpoco
Copy link
Contributor

@Shunpoco Shunpoco commented Dec 31, 2024

This resolves a part of #116971 .

The directory tests/mir-opt/issues has only one test issue_75439.rs which should add FileCheck annotations.

Originally it was introduced in #75580 to confirm that there were duplicated basic blocks against or-patterns, but in #123067 the duplication was resolved. So FileCheck should ensure that there is no such duplication any more.

@rustbot
Copy link
Collaborator

rustbot commented Dec 31, 2024

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @Mark-Simulacrum (or someone else) some time within the next two weeks.

Please see the contribution instructions for more information. Namely, in order to ensure the minimum review times lag, PR authors and assigned reviewers should ensure that the review label (S-waiting-on-review and S-waiting-on-author) stays updated, invoking these commands when appropriate:

  • @rustbot author: the review is finished, PR author should check the comments and take action accordingly
  • @rustbot review: the author is ready for a review, this PR will be queued again in the reviewer's queue

@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 Dec 31, 2024
@Shunpoco
Copy link
Contributor Author

r? @cjgillot

@rustbot rustbot assigned cjgillot and unassigned Mark-Simulacrum Dec 31, 2024
@Shunpoco Shunpoco marked this pull request as ready for review December 31, 2024 00:38
@Shunpoco Shunpoco force-pushed the 116971-mir-opt-issues branch from 5f9d9e5 to 9ec1fc0 Compare January 1, 2025 18:04
From rust-lang#123067, this test should ensure that there is no duplicated basic block against or-patterns.

Signed-off-by: Shunpoco <tkngsnsk313320@gmail.com>
@Shunpoco Shunpoco force-pushed the 116971-mir-opt-issues branch from 9ec1fc0 to 7061551 Compare January 4, 2025 12:53
@dianqk
Copy link
Member

dianqk commented Feb 28, 2025

r? DianQK
@bors r+ rollup=always

@bors
Copy link
Contributor

bors commented Feb 28, 2025

📌 Commit 7061551 has been approved by DianQK

It is now in the queue for this repository.

@rustbot rustbot assigned dianqk and unassigned cjgillot Feb 28, 2025
@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 Feb 28, 2025
jieyouxu added a commit to jieyouxu/rust that referenced this pull request Feb 28, 2025
…DianQK

Add FileCheck annotations to mir-opt/issues

This resolves a part of rust-lang#116971 .

The directory `tests/mir-opt/issues` has only one test issue_75439.rs which should add FileCheck annotations.

Originally it was introduced in rust-lang#75580 to confirm that there were duplicated basic blocks against or-patterns, but in rust-lang#123067 the duplication was resolved. So FileCheck should ensure that there is no such duplication any more.
jieyouxu added a commit to jieyouxu/rust that referenced this pull request Feb 28, 2025
…DianQK

Add FileCheck annotations to mir-opt/issues

This resolves a part of rust-lang#116971 .

The directory `tests/mir-opt/issues` has only one test issue_75439.rs which should add FileCheck annotations.

Originally it was introduced in rust-lang#75580 to confirm that there were duplicated basic blocks against or-patterns, but in rust-lang#123067 the duplication was resolved. So FileCheck should ensure that there is no such duplication any more.
bors added a commit to rust-lang-ci/rust that referenced this pull request Mar 1, 2025
Rollup of 10 pull requests

Successful merges:

 - rust-lang#134943 (Add FileCheck annotations to mir-opt/issues)
 - rust-lang#137017 (Don't error when adding a staticlib with bitcode files compiled by newer LLVM)
 - rust-lang#137197 (Update some comparison codegen tests now that they pass in LLVM20)
 - rust-lang#137540 (Fix (more) test directives that were accidentally ignored)
 - rust-lang#137551 (import `simd_` intrinsics)
 - rust-lang#137599 (tests: use minicore more)
 - rust-lang#137673 (Fix Windows `Command` search path bug)
 - rust-lang#137676 (linker: Fix escaping style for response files on Windows)
 - rust-lang#137693 (Re-enable `--generate-link-to-defintion` for tools internal rustdoc)
 - rust-lang#137770 (Fix sized constraint for unsafe binder)

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit d9c34a7 into rust-lang:master Mar 1, 2025
6 checks passed
@rustbot rustbot added this to the 1.87.0 milestone Mar 1, 2025
rust-timer added a commit to rust-lang-ci/rust that referenced this pull request Mar 1, 2025
Rollup merge of rust-lang#134943 - Shunpoco:116971-mir-opt-issues, r=DianQK

Add FileCheck annotations to mir-opt/issues

This resolves a part of rust-lang#116971 .

The directory `tests/mir-opt/issues` has only one test issue_75439.rs which should add FileCheck annotations.

Originally it was introduced in rust-lang#75580 to confirm that there were duplicated basic blocks against or-patterns, but in rust-lang#123067 the duplication was resolved. So FileCheck should ensure that there is no such duplication any more.
@Shunpoco Shunpoco deleted the 116971-mir-opt-issues branch March 1, 2025 17:15
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.

6 participants