Skip to content

Sema: Downgrade potentially unavailable enum cases to a warning in module interfaces [5.5] #37870

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

There are two pieces here:

  • A -warn-on-potentially-unavailable-enum-case flag is passed down by
    the driver when producing a swiftinterface

  • When consuming a swiftinterface, also enable this behavior

Part of rdar://problem/78306593.

…dule interfaces

There are two pieces here:

- A -warn-on-potentially-unavailable-enum-case flag is passed down by
  the driver when *producing* a swiftinterface

- When *consuming* a swiftinterface, also enable this behavior

Part of rdar://problem/78306593.
@slavapestov
Copy link
Contributor Author

swiftlang/swift-driver#708
@swift-ci Please test

@swift-ci
Copy link
Contributor

Build failed
Swift Test Linux Platform
Git Sha - 2fb3f6d

@swift-ci
Copy link
Contributor

Build failed
Swift Test OS X Platform
Git Sha - 2fb3f6d

@slavapestov
Copy link
Contributor Author

swiftlang/swift-driver#708
@swift-ci Please test

@swift-ci
Copy link
Contributor

Build failed
Swift Test OS X Platform
Git Sha - 2fb3f6d

@slavapestov
Copy link
Contributor Author

swiftlang/swift-driver#708
@swift-ci Please test macOS

@slavapestov
Copy link
Contributor Author

swiftlang/swift-driver#708
@swift-ci Please test Windows

@slavapestov slavapestov merged commit d79b69b into swiftlang:release/5.5 Jun 15, 2021
@AnthonyLatsis AnthonyLatsis added 🍒 release cherry pick Flag: Release branch cherry picks swift 5.5 labels Jan 8, 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.5
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants