Skip to content

[5.1][CSSimplify] Reject key path if root type is AnyObject #24026

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

xedin
Copy link
Contributor

@xedin xedin commented Apr 14, 2019

Detect situations where AnyObject is attempted to be used as a root type of the key path
early and diagnose via new diagnostics framework.

Resolves: rdar://problem/49413561

(cherry picked from commit 072e84a)

Detect situations where `AnyObject` is attempted to be used as a root type of the key path
early and diagnose via new diagnostics framework.

Resolves: rdar://problem/49413561

(cherry picked from commit 072e84a)
@xedin
Copy link
Contributor Author

xedin commented Apr 14, 2019

@swift-ci please test

@xedin
Copy link
Contributor Author

xedin commented Apr 14, 2019

@swift-ci please test source compatibility

@swift-ci
Copy link
Contributor

Build failed
Swift Test OS X Platform
Git Sha - 9be7646

@xedin
Copy link
Contributor Author

xedin commented Apr 14, 2019

@swift-ci please test macOS platform

@swift-ci
Copy link
Contributor

Build failed
Swift Test OS X Platform
Git Sha - 9be7646

@xedin
Copy link
Contributor Author

xedin commented Apr 14, 2019

Ah right, I need to cherry-pick changes to fixMemberRef as well.

… declaration to have a fix

We should be able to attach fixes to overload choices that don't
refer to declarations e.g. keypath applications which refer to
`AnyObject` is a base type.

(cherry picked from commit 374a10c)
@xedin
Copy link
Contributor Author

xedin commented Apr 14, 2019

@swift-ci please test

@swift-ci
Copy link
Contributor

Build failed
Swift Test OS X Platform
Git Sha - 9be7646

@xedin
Copy link
Contributor Author

xedin commented Apr 14, 2019

@swift-ci please test source compatibility

@swift-ci
Copy link
Contributor

Build failed
Swift Test Linux Platform
Git Sha - 9be7646

@xedin
Copy link
Contributor Author

xedin commented Apr 14, 2019

@swift-ci please test macOS platform

@xedin xedin merged commit 9740a31 into swiftlang:swift-5.1-branch Apr 15, 2019
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