Overlay: add C++ interop libraries #83347
Conversation
compnerd
requested review from
etcwilde,
edymtt and
justice-adams-apple
as code owners
|
@swift-ci please smoke test |
|
@swift-ci please smoke test |
|
Please test with following PRs: @swift-ci please build toolchain Windows platform |
|
@swift-ci please smoke test macOS platform |
compnerd
force-pushed
the
c++interop
branch
2 times, most recently
from
ab44174 to
61906d0
Compare
|
@swift-ci please smoke test windows platform |
|
@swift-ci please smoke test |
|
Please test with following PRs: @swift-ci please build toolchain Windows platform |
|
@swift-ci please smoke test |
|
@swift-ci please smoke test macOS platform |
|
@swift-ci please smoke test macOS platform |
This introduces a build of the C++ interop runtime support to the new runtime library build.
|
@swift-ci please smoke test |
|
@swift-ci please test Windows platform |
Co-authored-by: Eric Miotto <[email protected]>
|
@swift-ci please smoke test |
|
@swift-ci please smoke test macOS platform |
2 similar comments
|
@swift-ci please smoke test macOS platform |
|
@swift-ci please smoke test macOS platform |
| if(NOT APPLE) | ||
| add_subdirectory(cxxshim) | ||
| endif() | ||
| if(LINUX) |
There was a problem hiding this comment.
I actually need to target a Linux that does not have a libstdc++, but uses libcxx instead. This assumption is incorrect.
There was a problem hiding this comment.
Fair enough - I think that we can iterate on that subsequently.
| if(NOT APPLE) | ||
| add_subdirectory(cxxshim) | ||
| endif() |
There was a problem hiding this comment.
This doesn't seem correct: CxxShim headers are used on all of the platforms, including Apple ones.
This introduces a build of the C++ interop runtime support to the new runtime library build.