Skip to content

Fix ICE for while loop with assignment condition with LHS place expr - #112392

Merged
bors merged 1 commit into
rust-lang:masterfrom
jieyouxu:issue-112385
Jun 8, 2023
Merged

Fix ICE for while loop with assignment condition with LHS place expr#112392
bors merged 1 commit into
rust-lang:masterfrom
jieyouxu:issue-112385

Conversation

@jieyouxu

@jieyouxujieyouxu commented Jun 7, 2023

Copy link
Copy Markdown
Member

Fixes#112385.

@rustbot

Copy link
Copy Markdown
Collaborator

r? @eholk

(rustbot has picked a reviewer for you, use r? to override)

@rustbotrustbot 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 Jun 7, 2023
Comment threadtests/ui/typeck/issue-112385.rs Outdated

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This could use a better UI test title...

Copy link
Copy Markdown
MemberAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I updated the test name but it seems GitHub is choking with PRs or something...

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

https://www.githubstatus.com/ seems like a github problem, i will approve it when the PR syncs. Thanks for updating.

@jieyouxu
jieyouxuforce-pushed the issue-112385 branch 2 times, most recently from 1127c58 to 4b4a1e6CompareJune 7, 2023 18:15
@rust-log-analyzer

This comment has been minimized.

@compiler-errors

Copy link
Copy Markdown
Contributor

@bors r+ rollup

@bors

bors commented Jun 7, 2023

Copy link
Copy Markdown
Collaborator

📌 Commit adbfd0d has been approved by compiler-errors

It is now in the queue for this repository.

@borsbors 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 Jun 7, 2023
bors added a commit to rust-lang-ci/rust that referenced this pull request Jun 8, 2023
…llaumeGomez
Rollup of 9 pull requests
Successful merges:
- rust-lang#112034 (Migrate `item_opaque_ty` to Askama)
- rust-lang#112179 (Avoid passing --cpu-features when empty)
- rust-lang#112309 (bootstrap: remove dependency `is-terminal`)
- rust-lang#112388 (Migrate GUI colors test to original CSS color format)
- rust-lang#112389 (Add a test for rust-lang#105709)
- rust-lang#112392 (Fix ICE for while loop with assignment condition with LHS place expr)
- rust-lang#112394 (Remove accidental comment)
- rust-lang#112396 (Track more diagnostics in `rustc_expand`)
- rust-lang#112401 (Don't `use compile_error as print`)
r? `@ghost`
`@rustbot` modify labels: rollup
@bors
bors merged commit 80829ce into rust-lang:masterJun 8, 2023
@rustbotrustbot added this to the 1.72.0 milestone Jun 8, 2023
@jieyouxu
jieyouxu deleted the issue-112385 branch October 22, 2024 12:35
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-borsStatus: Waiting on bors to run and complete tests. Bors will change the label on completion.T-compilerRelevant 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.

Internal compiler error instead of a reminder that pattern-matching in loops uses while let or a note that some variable identifiers are ambiguous

6 participants

@jieyouxu@rustbot@rust-log-analyzer@compiler-errors@bors@eholk