[libcxx-commits] [PATCH] D89041: [libc++] Include <__config_site> from <__config>

[Louis Dionne via Phabricator via libcxx-commits]

ldionne added inline comments.


================
Comment at: libcxx/CMakeLists.txt:403
 set(LIBCXX_BINARY_DIR  ${CMAKE_CURRENT_BINARY_DIR})
 set(LIBCXX_BINARY_INCLUDE_DIR "${LIBCXX_BINARY_DIR}/include/c++build")
+set(LIBCXX_GENERATED_INCLUDE_DIR "${LIBCXX_BINARY_DIR}/include/c++/v1")
----------------
phosek wrote:
> Do we still need this? Could we copy ABI files directly into `LIBCXX_GENERATED_INCLUDE_DIR`?
I wanted to tackle that as a separate issue after investigating.

I actually don't see why we need to copy them at all, since the libc++abi headers don't need a generation step.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D89041