Skip to content

[sil] Extract out ApplySite/FullApplySite into their own header. #19542

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

gottesmm
Copy link
Contributor

I believe that these were in SILInstruction for historic reasons. This is a
separate API on top of SILInstruction so it makes sense to pull it out into its
own header.

I believe that these were in SILInstruction for historic reasons. This is a
separate API on top of SILInstruction so it makes sense to pull it out into its
own header.
@gottesmm gottesmm requested a review from atrick September 25, 2018 20:33
@gottesmm
Copy link
Contributor Author

@swift-ci smoke test

@atrick
Copy link
Contributor

atrick commented Sep 25, 2018

👍

@gottesmm gottesmm merged commit a23135a into swiftlang:master Sep 25, 2018
@gottesmm gottesmm deleted the pr-941d42ff9486902634d583f9ca6ffc2361933df5 branch September 25, 2018 23:29
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