Skip to content

Conversation

aschwaighofer
Copy link
Contributor

@aschwaighofer aschwaighofer commented Mar 15, 2018

  • Description: Generic classes with protocol meta type stored properties crash during initialization because we ignored the second word of storage of existential meta types.

  • Scope: This regression was introduced because we more often initialize generic class metadata which exercises the code path the causes the crash.

  • Testing: CI test added.

  • Risk: Low

  • Reviewed: Joe G.

rdar://37924505

@aschwaighofer
Copy link
Contributor Author

@swift-ci Please test

@aschwaighofer
Copy link
Contributor Author

@swift-ci Please nominate

Copy link
Contributor

@jckarter jckarter left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, thanks Arnold!

@swift-ci
Copy link
Contributor

Build failed
Swift Test Linux Platform
Git Sha - 22cb5bc

@swift-ci
Copy link
Contributor

Build failed
Swift Test OS X Platform
Git Sha - 22cb5bc

@aschwaighofer
Copy link
Contributor Author

@swift-ci Please test

@swift-ci
Copy link
Contributor

Build failed
Swift Test Linux Platform
Git Sha - 22cb5bc

@swift-ci
Copy link
Contributor

Build failed
Swift Test OS X Platform
Git Sha - 22cb5bc

@tkremenek tkremenek merged commit e5e5a80 into swiftlang:swift-4.1-branch Mar 15, 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.

4 participants