-
Notifications
You must be signed in to change notification settings - Fork 10.5k
[cxx-interop][nfc] Add a ClangImporter request zone. #39435
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
[cxx-interop][nfc] Add a ClangImporter request zone. #39435
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, just a couple of questions and some nits
Thank you for the review @hamishknight! |
50997f0
to
9d5232d
Compare
@swift-ci please smoke test. |
Any other review comments? OK if I land this? |
This is just the boilerplate for adding a request zone. I haven't actually added any requrests in this commit.
The whole file was indented by one space.
9d5232d
to
d52973f
Compare
@swift-ci please smoke test and merge. |
@swift-ci please smoke test Linux. |
@swift-ci please test Windows. |
1 similar comment
@swift-ci please test Windows. |
This is a nfc patch. I've seen it pass on the Windows CI before. I'm just going to land it. |
This is just the boilerplate for adding a request zone. I haven't actually added any requrests in this commit.