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 one more case to avoid ICE #94355

Merged
merged 1 commit into from
Feb 26, 2022
Merged

Add one more case to avoid ICE #94355

merged 1 commit into from
Feb 26, 2022

Conversation

ouz-a
Copy link
Contributor

@ouz-a ouz-a commented Feb 25, 2022

Fix for the #94291, added one more case to related function to avoid ICE.

Not sure if my test is in the correct place 😅

@rustbot rustbot added the T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. label Feb 25, 2022
@rust-highfive
Copy link
Collaborator

r? @michaelwoerister

(rust-highfive has picked a reviewer for you, use r? to override)

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Feb 25, 2022
@michaelwoerister
Copy link
Member

Thanks for the PR, @ouz-a! The test location looks good to me -- however, I don't know this code, so I'm going to re-assign.
r? rust-lang/compiler

@oli-obk
Copy link
Contributor

oli-obk commented Feb 25, 2022

@bors r+ rollup

@bors
Copy link
Contributor

bors commented Feb 25, 2022

📌 Commit 8cbd6e825b303d591af5e3b743a100b7b2ed077d has been approved by oli-obk

@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 25, 2022
@rust-log-analyzer

This comment has been minimized.

@rust-log-analyzer

This comment has been minimized.

@oli-obk
Copy link
Contributor

oli-obk commented Feb 25, 2022

@bors r+ rollup

@bors
Copy link
Contributor

bors commented Feb 25, 2022

📌 Commit a625de270db5e582a88fb2a0f11d8d5926fa5ac0 has been approved by oli-obk

@Mark-Simulacrum
Copy link
Member

Squashed commits -- please try to keep the history a little more tidy in the future.

@bors r=oli-bok

@bors
Copy link
Contributor

bors commented Feb 25, 2022

📌 Commit 8c96487 has been approved by oli-bok

matthiaskrgr added a commit to matthiaskrgr/rust that referenced this pull request Feb 25, 2022
Add one more case to avoid ICE

Fix for the  rust-lang#94291, added one more case to related function to avoid ICE.

Not sure if my test is in the correct place 😅
bors added a commit to rust-lang-ci/rust that referenced this pull request Feb 26, 2022
…askrgr

Rollup of 5 pull requests

Successful merges:

 - rust-lang#93603 (Populate liveness facts when calling `get_body_with_borrowck_facts` without `-Z polonius`)
 - rust-lang#93870 (Fix switch on discriminant detection in a presence of coverage counters)
 - rust-lang#94355 (Add one more case to avoid ICE)
 - rust-lang#94363 (Remove needless borrows from core::fmt)
 - rust-lang#94377 (`check_used` should only look at actual `used` attributes)

Failed merges:

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit 5af6624 into rust-lang:master Feb 26, 2022
@rustbot rustbot added this to the 1.61.0 milestone Feb 26, 2022
@ouz-a
Copy link
Contributor Author

ouz-a commented Feb 26, 2022

Squashed commits -- please try to keep the history a little more tidy in the future.

@bors r=oli-bok

ah sorry about that

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.

8 participants