Skip to content

[const-expr] Teach ConstExpr how to handle switch_enum/unchecked_enum… #22757

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
Merged
merged 1 commit into from
Feb 21, 2019
Merged

[const-expr] Teach ConstExpr how to handle switch_enum/unchecked_enum… #22757

merged 1 commit into from
Feb 21, 2019

Conversation

gottesmm
Copy link
Contributor

…_data in ossa.

@gottesmm
Copy link
Contributor Author

@swift-ci smoke test and merge

@ravikandhadai
Copy link
Contributor

LGTM.

@gottesmm
Copy link
Contributor Author

@shahmishal the linux test failed due to I believe a change from the stable merge (or not cleaning). Are the bots cleaned?

@gottesmm
Copy link
Contributor Author

@swift-ci smoke test linux platform

@shahmishal
Copy link
Member

I triggered a clean build job, but you can request a clean build from PR testing too.

@gottesmm
Copy link
Contributor Author

@shahmishal happened again

@gottesmm
Copy link
Contributor Author

@swift-ci clean smoke test

@gottesmm gottesmm merged commit 4a1b98c into swiftlang:master Feb 21, 2019
@gottesmm gottesmm deleted the pr-6e89f4298efb94406a3ad6b55617ae741dfb65e0 branch February 21, 2019 01:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@marcrasi marcrasi marcrasi approved these changes

@ravikandhadai ravikandhadai ravikandhadai approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@gottesmm @ravikandhadai @shahmishal @marcrasi