Skip to content

[5.1 04-24-2019] [IRGen] Emit lazy opaque type descriptors. #24339

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

DougGregor
Copy link
Member

These were recorded but never emitted. Fixes rdar://problem/50258536.

@DougGregor DougGregor requested a review from a team as a code owner April 27, 2019 04:56
@DougGregor
Copy link
Member Author

@swift-ci please test

@DougGregor
Copy link
Member Author

@AnnaZaks FYI

@swift-ci
Copy link
Contributor

Build failed
Swift Test Linux Platform
Git Sha - 899a120d38716c67d2e5754f7fe64ef9c275e213

@swift-ci
Copy link
Contributor

Build failed
Swift Test OS X Platform
Git Sha - 899a120d38716c67d2e5754f7fe64ef9c275e213

These were recorded but never emitted. Fixes rdar://problem/50258536.
@DougGregor DougGregor force-pushed the irgen-emit-lazy-opaque-descriptors-5.1-04-24-2019 branch from 899a120 to ca2fc94 Compare April 27, 2019 16:54
@DougGregor
Copy link
Member Author

@swift-ci please test

@swift-ci
Copy link
Contributor

Build failed
Swift Test Linux Platform
Git Sha - 899a120d38716c67d2e5754f7fe64ef9c275e213

@swift-ci
Copy link
Contributor

Build failed
Swift Test OS X Platform
Git Sha - 899a120d38716c67d2e5754f7fe64ef9c275e213

@DougGregor
Copy link
Member Author

@swift-ci please test

@DougGregor
Copy link
Member Author

@swift-ci please test Linux

@swift-ci
Copy link
Contributor

Build failed
Swift Test Linux Platform
Git Sha - ca2fc94

@DougGregor
Copy link
Member Author

@swift-ci please test Linux

@DougGregor DougGregor merged commit 951c132 into swiftlang:swift-5.1-branch-04-24-2019 Apr 29, 2019
@DougGregor DougGregor deleted the irgen-emit-lazy-opaque-descriptors-5.1-04-24-2019 branch April 29, 2019 18:44
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