[libcxx-commits] [PATCH] D82702: [libc++] Use a proper CMake target to represent libc++ headers

[Petr Hosek via Phabricator via libcxx-commits]

phosek added a comment.

In D82702#2146536 <https://reviews.llvm.org/D82702#2146536>, @phosek wrote:

> One more thing I've noticed is that both of those invocations include `__config_site` which is added by the `cxx-headers` interface library, so that library must be used, it's just as if `target_include_directories(${CXX_HEADER_TARGET} INTERFACE ${output_dir})` was ignored when building the instrumented version.


Another thing I just found out, this is only affecting paths that end with `include/c++/v1`, if I add any other path like `include/c++/v2`, it's works fine. So it seems likes something is filtering out paths that end with `include/c++/v1` from include directories. I'm starting to suspect a bug in CMake (I'm using version 3.16.3 for reference).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D82702