Skip to content

fix SR-4166: __mode__ attribute presence check changed. #7923

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

gonzalolarralde
Copy link
Contributor

@gonzalolarralde gonzalolarralde commented Mar 6, 2017

Changes how the __mode__ attribute is checked. __has_attribute should query for __mode__, assuming the DI variant will be available.

Resolves SR-4166.

@hughbe Would you be able to confirm if the original intention of this line is still covered and working fine in your environment? Thanks.

@hughbe
Copy link
Contributor

hughbe commented Mar 6, 2017

@swift-ci please smoke test

@hughbe
Copy link
Contributor

hughbe commented Mar 6, 2017

Looks like this works: https://godbolt.org/g/H2hmUp.

Thanks!!

@hughbe hughbe merged commit 3f6ede7 into swiftlang:master Mar 6, 2017
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.

2 participants