[libcxx-commits] [PATCH] D65831: libc++: add `-include` or `/FI` to the interface compile definitions

[Shoaib Meenai via Phabricator via libcxx-commits]

smeenai requested changes to this revision.
smeenai added a comment.
This revision now requires changes to proceed.

When libc++ is installed, the `__config_site` and `__config` are concatenated to make a single `__config`. I'm imagining this is specific to using libc++ from the build tree? If so, you should use `INTERFACE_INCLUDE_DIRECTORIES` and `$<BUILD_INTERFACE:...>` to limit it to builds.


Repository:
  rCXX libc++

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

https://reviews.llvm.org/D65831