[PATCH] D101458: [libc++] Do not copy the libc++abi headers into the libc++ build tree

[Louis Dionne via Phabricator via llvm-commits]

ldionne added inline comments.


================
Comment at: llvm/utils/gn/secondary/libcxx/include/BUILD.gn:271-281
-    if (target_os != "mac" && target_os != "win") {
-      # libcxx/cmake/Modules/HandleLibCXXABI.cmake sets
-      # LIBCXX_CXX_ABI_HEADER_TARGET if the libcxx abi library either of
-      # "libstdc++", "libsupc++", "libcxxabi", "libcxxrt", but not if it's "none",
-      # "default", or "vcruntime". So on Windows, these don't get copied due to
-      # LIBCXX_CXX_ABI_HEADER_TARGET not being set.
-      # On macOS, libcxx/CMakeLists.txt sets LIBCXX_CXX_ABI_SYSTEM to 1, which
----------------
Trying to save you a headache @thakis, but I'd like you to double-check this is OK.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D101458/new/

https://reviews.llvm.org/D101458