Skip to content

Fix enum element constructor call with trailing closure [5.0] #21927

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

Conversation

slavapestov
Copy link
Contributor

We use TupleShuffleExpr in RValue position for enum element payloads, and
it can happen that the source is a scalar if you're calling the enum element
constructor with a trailing closure.

This reverts commit 7960660.

Fixes https://bugs.swift.org/browse/SR-9675.

@slavapestov slavapestov requested a review from a team as a code owner January 16, 2019 20:08
@slavapestov
Copy link
Contributor Author

@swift-ci Please test

@swift-ci
Copy link
Contributor

Build failed
Swift Test Linux Platform
Git Sha - d1455b2e880f6695a0fed5e02cb2566655b28d5e

@swift-ci
Copy link
Contributor

Build failed
Swift Test OS X Platform
Git Sha - d1455b2e880f6695a0fed5e02cb2566655b28d5e

Andreas Jönsson and others added 3 commits January 17, 2019 15:49
…h can't have scalar-to-tuple or tuple-to-scalar"

We use TupleShuffleExpr in RValue position for enum element payloads, and
it can happen that the source is a scalar if you're calling the enum element
constructor with a trailing closure.

This reverts commit 7960660.

Fixes <https://bugs.swift.org/browse/SR-9675>.
@slavapestov slavapestov force-pushed the enum-element-trailing-closure-5.0 branch from d1455b2 to 1e8a0ba Compare January 17, 2019 21:18
@slavapestov
Copy link
Contributor Author

@swift-ci Please test

@swift-ci
Copy link
Contributor

Build failed
Swift Test Linux Platform
Git Sha - d1455b2e880f6695a0fed5e02cb2566655b28d5e

@swift-ci
Copy link
Contributor

Build failed
Swift Test OS X Platform
Git Sha - d1455b2e880f6695a0fed5e02cb2566655b28d5e

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.

3 participants