Skip to content

Treat irrefutable casts as irrefutable patterns. #9449

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 10, 2017

Conversation

CodaFi
Copy link
Contributor

@CodaFi CodaFi commented May 10, 2017

In response to a post on Swift-dev by @pushkarnk that mentioned a regression in exhaustiveness analysis because irrefutable casts were not treated as irrefutable patterns. This patch does just that.

@CodaFi CodaFi requested a review from jrose-apple May 10, 2017 00:54
@CodaFi
Copy link
Contributor Author

CodaFi commented May 10, 2017

@swift-ci please test

@slavapestov
Copy link
Contributor

Are you planning on getting this into swift-4.0-branch also?

@swift-ci
Copy link
Contributor

Build failed
Jenkins build - Swift Test Linux Platform
Git Commit - 95a409566ce85cb0f7e0ac4a84e744a612e45ce2
Test requested by - @CodaFi

@CodaFi
Copy link
Contributor Author

CodaFi commented May 10, 2017

@slavapestov Yes.

@swift-ci
Copy link
Contributor

Build failed
Jenkins build - Swift Test OS X Platform
Git Commit - 95a409566ce85cb0f7e0ac4a84e744a612e45ce2
Test requested by - @CodaFi

@CodaFi CodaFi force-pushed the the-rISA branch 2 times, most recently from 5a6baf4 to f57ee82 Compare May 10, 2017 04:45
@CodaFi
Copy link
Contributor Author

CodaFi commented May 10, 2017

@swift-ci please smoke test and merge

@CodaFi
Copy link
Contributor Author

CodaFi commented May 10, 2017

@swift-ci please smoke test

@CodaFi
Copy link
Contributor Author

CodaFi commented May 10, 2017

⛵️

@CodaFi CodaFi merged commit 0a0ab12 into swiftlang:master May 10, 2017
@CodaFi CodaFi deleted the the-rISA branch May 10, 2017 16:10
Copy link
Contributor

@jrose-apple jrose-apple left a comment

Choose a reason for hiding this comment

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

Thank you! Looks good.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants