Skip to content

[5.2][Stdlib] Eagerly realize EmptyDictionarySingleton and EmptySetSingleton. #29849

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

mikeash
Copy link
Contributor

@mikeash mikeash commented Feb 14, 2020

Cherry-pick #29848 to 5.2.

These objects can escape into ObjC without their class being realized first, which can cause a crash if the unrealized class gets passed into the ObjC runtime.

rdar://problem/59295395

These objects can escape into ObjC without their class being realized first, which can cause a crash if the unrealized class gets passed into the ObjC runtime.

rdar://problem/59295395
@mikeash mikeash requested a review from Catfish-Man February 14, 2020 16:59
@mikeash mikeash requested a review from a team as a code owner February 14, 2020 16:59
@mikeash
Copy link
Contributor Author

mikeash commented Feb 14, 2020

@swift-ci please test

@swift-ci
Copy link
Contributor

Build failed
Swift Test Linux Platform
Git Sha - 2c86c61

@mikeash
Copy link
Contributor Author

mikeash commented Feb 14, 2020

@swift-ci please test linux platform

@swift-ci
Copy link
Contributor

Build failed
Swift Test Linux Platform
Git Sha - 2c86c61

@mikeash
Copy link
Contributor Author

mikeash commented Feb 15, 2020

@swift-ci please test linux platform

@swift-ci
Copy link
Contributor

Build failed
Swift Test Linux Platform
Git Sha - 2c86c61

@benlangmuir
Copy link
Contributor

@swift-ci please test linux platform

@swift-ci
Copy link
Contributor

Build failed
Swift Test Linux Platform
Git Sha - 2c86c61

@mikeash
Copy link
Contributor Author

mikeash commented Feb 15, 2020

@swift-ci please test linux platform

@mikeash mikeash merged commit 1933cc7 into swiftlang:swift-5.2-branch Feb 18, 2020
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.

5 participants