[libcxx-commits] [PATCH] D132526: [libcxxabi] Unbreak `LLVM_ENABLE_RUNTIMES=libcxxabi` build v2

[Petr Hosek via Phabricator via libcxx-commits]

phosek added inline comments.


================
Comment at: libcxxabi/cmake/Modules/HandleLibCXX.cmake:4-6
+option(LIBCXXABI_USE_PREBUILT_LIBCXX_HEADERS
+    "Use prebuilt libc++ headers because we are not also building libcc++."
+    OFF)
----------------
Ericson2314 wrote:
> My `TARGET cxx-headers` trick didn't work because that target isn't defined until afterwords, since `libcxx` mostly depends on`libcxxabi`.
Could we use `LIBCXXABI_LIBCXX_INCLUDE` instead of introducing another option? That is, if `LIBCXXABI_LIBCXX_INCLUDE` is not empty, it implies using prebuilt libc++ headers.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D132526