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.

Sign up for GitHub

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

Jump to bottom

Rollup of 3 pull requests #122648

Closed
wants to merge 6 commits into from
Closed

Rollup of 3 pull requests #122648

wants to merge 6 commits into from

Conversation

matthiaskrgr
Copy link
Member

Zalathar and others added 6 commits March 17, 2024 22:41
@Zalathar
Mention Zalathar for coverage changes
36f8d67
@the8472
add test for rust-lang#122301 to cover behavior that's on stable
fefd06d 
if this ought to be broken it should at least happen intentionally
@saethlin
Remove some only- clauses from mir-opt tests
6828f0c
@matthiaskrgr
Rollup merge of rust-lang#122572 - the8472:test-const-deadness, r=Ral…
ce707de 
…fJung

add test for rust-lang#122301 to cover behavior that's on stable

If this ought to be broken it should at least happen intentionally

See rust-lang#122301
@matthiaskrgr
Rollup merge of rust-lang#122615 - Zalathar:triagebot, r=Mark-Simulacrum
e74b798 
Mention Zalathar for coverage changes

Hopefully I don't regret all the extra notifications. 🤞
@matthiaskrgr
Rollup merge of rust-lang#122645 - saethlin:portable-mir-opt-tests, r…
dd38f83 
…=Nadrieril

Remove some only- clauses from mir-opt tests

Derived from rust-lang#122295

Many of these tests were originally codegen tests, and MIR is more trivially portable than LLVM IR. We simply don't need to restrict the platform in most cases.

r? Nadrieril
@rustbot rustbot added A-meta Area: Issues & PRs about the rust-lang/rust repository itself 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. rollup A PR which is a rollup labels Mar 17, 2024
@matthiaskrgr
Copy link
Member Author

@bors r+ rollup=never p=3

@bors
Copy link
Contributor

bors commented Mar 17, 2024

📌 Commit dd38f83 has been approved by matthiaskrgr

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 Mar 17, 2024
@matthiaskrgr matthiaskrgr deleted the rollup-2k9yg2c branch September 1, 2024 17:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
A-meta Area: Issues & PRs about the rust-lang/rust repository itself rollup A PR which is a rollup 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
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@matthiaskrgr @bors @rustbot @Zalathar @the8472 @saethlin