Skip to content

Sema: Diagnose @_backDeploy attributes on initializers and deinitializers #59283

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

Conversation

tshortli
Copy link
Contributor

@tshortli tshortli commented Jun 6, 2022

Applying the @_backDeploy attribute to an initializer currently causes a duplicate symbol error at link time so for now reject it on those declarations for clarity. Also, reject it on deinitializers as well since the attribute cannot have the desired effect on deinit.

Resolves rdar://94450734

…lizers.

Applying the `@_backDeploy` attribute to an initializer currently causes a duplicate symbol error at link time so for now reject it on those declarations for clarity. Also, reject it on deinitializers as well since the attribute cannot have the desired effect on `deinit`.

Resolves rdar://94450734
@tshortli
Copy link
Contributor Author

tshortli commented Jun 6, 2022

@swift-ci please smoke test

@tshortli tshortli merged commit e63df1e into swiftlang:main Jun 6, 2022
@tshortli tshortli deleted the diagnose-back-deploy-on-init-deinit branch June 6, 2022 22:50
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.

1 participant