[libcxx-commits] [PATCH] D114248: [libc++] Improve CMake include directory search.

[Mark de Wever via Phabricator via libcxx-commits]

Mordante added inline comments.


================
Comment at: libcxx/src/CMakeLists.txt:200
   add_library(cxx_shared SHARED ${exclude_from_all} ${LIBCXX_SOURCES} ${LIBCXX_HEADERS})
+  target_include_directories(cxx_shared PRIVATE ${CMAKE_CURRENT_SOURCE_DIR})
   target_link_libraries(cxx_shared PUBLIC cxx-headers
----------------
ldionne wrote:
> What are these useful for?
Basically it's based on the comment https://reviews.llvm.org/D70631#inline-1071881. In that original version I used the blunt tool of `include_directories(${CMAKE_CURRENT_SOURCE_DIR})`. Without this the `ryu` related headers can't be found.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D114248