Skip to content

Support concrete type propagation into witness_method instructions only for directly inherited conformances for now #11708

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
merged 1 commit into from
Aug 31, 2017

Conversation

swiftix
Copy link
Contributor

@swiftix swiftix commented Aug 31, 2017

Indirectly inherited conformances cannot be properly handled yet due to some representation issues with init_existential_* instructions.

This fixes a compiler crash reported in rdar://34022255.

…ly for directly inherited conformances for now

Indirectly inherited conformances cannot be properly handled yet due to some representation issues with init_existential_* instructions.

This fixes a compiler crash reported in rdar://34022255.
@swiftix
Copy link
Contributor Author

swiftix commented Aug 31, 2017

Explanation: Fixes a compiler crash in the sil-combine pass
Risk: Minimal, as it reduces the applicability of an existing optimization.
Reviewed By: Doug Gregor
Testing: A new unit test was added.
Bug:
Radar: rdar://problem/34022255

@swiftix swiftix requested a review from DougGregor August 31, 2017 16:54
@swiftix
Copy link
Contributor Author

swiftix commented Aug 31, 2017

@DougGregor Doug, could you review for CCC? This is what we discussed yesterday.

@swiftix
Copy link
Contributor Author

swiftix commented Aug 31, 2017

@swift-ci please test

Copy link
Member

@DougGregor DougGregor left a comment

Choose a reason for hiding this comment

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

LGTM!

@swiftix swiftix merged commit 148b57b into swiftlang:swift-4.0-branch Aug 31, 2017
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