Skip to content

[5.7][CSClosure] Fix crash in fallthrough statement checking #59083

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

Merged
merged 1 commit into from
May 26, 2022

Conversation

xedin
Copy link
Contributor

@xedin xedin commented May 25, 2022

Cherry-pick of #59059


fallthrough requires both source and destination case
"preambles" be type-checked before it could be validated,
which means that solution application for case statements
in a switch has to be split in two - preamble first and
bodies afterwards.

Resolves: #59035
Resolves: rdar://93796211
(cherry picked from commit 4e9992c)

`fallthrough` requires both source and destination `case`
"preambles" be type-checked before it could be validated,
which means that solution application for `case` statements
in a `switch` has to be split in two - preamble first and
bodies afterwards.

Resolves: swiftlang#59035
Resolves: rdar://93796211
(cherry picked from commit 4e9992c)
@xedin xedin requested a review from hborla May 25, 2022 20:21
@xedin xedin requested a review from a team as a code owner May 25, 2022 20:21
@xedin xedin added the r5.7 label May 25, 2022
@xedin
Copy link
Contributor Author

xedin commented May 25, 2022

@swift-ci please test

@xedin xedin merged commit 8e29f27 into swiftlang:release/5.7 May 26, 2022
@AnthonyLatsis AnthonyLatsis added 🍒 release cherry pick Flag: Release branch cherry picks swift 5.7 labels Jan 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🍒 release cherry pick Flag: Release branch cherry picks swift 5.7
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants