-
Notifications
You must be signed in to change notification settings - Fork 10.4k
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
build: enable control over the C++ interop modules #65398
Conversation
@swift-ci please test |
@swift-ci please build toolchain windows platform |
Thanks, Saleem! This looks great. |
@swift-ci please test |
0924c36
to
40bbed1
Compare
@swift-ci please test |
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.
The change looks good to me -- I'm doing some additional testing to make sure this works in other build setups.
Treat the C++ interop modules similar to the Concurrency and StringProcessing modules, enabled through a top-level global flag. The flag now enables disabling the C++ interop modules when the standard library is built. Based on the original work by Egor Zhdan!
40bbed1
to
f116173
Compare
@swift-ci please test |
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.
I confirm there are no regression in the additional testing I did, so this can be merged at your convenience.
@compnerd could you please cherry-pick this to release/5.9
as well?
…EROP=NO` This fixes the Swift LSAN CI job Linux (https://ci.swift.org/job/oss-swift-RA-lsan-linux-ubuntu-18_04/) that started failing after #65398. rdar://108674065
…EROP=NO` This fixes the Swift LSAN CI job Linux (https://ci.swift.org/job/oss-swift-RA-lsan-linux-ubuntu-18_04/) that started failing after #65398. rdar://108674065 (cherry picked from commit b586604)
Treat the C++ interop modules similar to the Concurrency and StringProcessing modules, enabled through a top-level global flag. The flag now enables disabling the C++ interop modules when the standard library is built.
Based on the original work by Egor Zhdan!