Skip to content

SIL: Generate external key path references with local candidate components. #17802

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

jckarter
Copy link
Contributor

@jckarter jckarter commented Jul 6, 2018

The other side of #17404. Since we don't want to generate up front key path metadata for properties/subscripts with no withheld implementation details, the client should generate a key path component that can be used to represent a key path component based on its public interface.

@jckarter jckarter force-pushed the external-component-local-attempt branch from ca3481a to 1ed556f Compare July 6, 2018 20:58
@jckarter
Copy link
Contributor Author

jckarter commented Jul 6, 2018

@swift-ci Please test

@swift-ci
Copy link
Contributor

swift-ci commented Jul 6, 2018

Build failed
Swift Test OS X Platform
Git Sha - 1ed556f02d1c3f147718db9a07e0a9593d2d8fc1

@jckarter jckarter force-pushed the external-component-local-attempt branch from 1ed556f to 0487f47 Compare July 6, 2018 21:08
@jckarter
Copy link
Contributor Author

jckarter commented Jul 6, 2018

@swift-ci Please test

@swift-ci
Copy link
Contributor

swift-ci commented Jul 6, 2018

Build failed
Swift Test OS X Platform
Git Sha - 1ed556f02d1c3f147718db9a07e0a9593d2d8fc1

@swift-ci
Copy link
Contributor

swift-ci commented Jul 6, 2018

Build failed
Swift Test Linux Platform
Git Sha - 1ed556f02d1c3f147718db9a07e0a9593d2d8fc1

…nents.

The other side of swiftlang#17404. Since we don't want to generate up front key path metadata for properties/subscripts with no withheld implementation details, the client should generate a key path component that can be used to represent a key path component based on its public interface.
@jckarter jckarter force-pushed the external-component-local-attempt branch from 0487f47 to 849d939 Compare July 6, 2018 21:24
@jckarter
Copy link
Contributor Author

jckarter commented Jul 6, 2018

@swift-ci Please test

@swift-ci
Copy link
Contributor

swift-ci commented Jul 6, 2018

Build failed
Swift Test Linux Platform
Git Sha - 0487f4737e5296e2c15488fc689e35b96027ddb4

@swift-ci
Copy link
Contributor

swift-ci commented Jul 6, 2018

Build failed
Swift Test OS X Platform
Git Sha - 0487f4737e5296e2c15488fc689e35b96027ddb4

@jckarter jckarter merged commit 296b9d1 into swiftlang:master Jul 6, 2018
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