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

[Louis Dionne via Phabricator via libcxx-commits]

ldionne added a comment.

@nickdesaulniers More generally, you must be setting `-DLLVM_ENABLE_PER_TARGET_RUNTIME_DIR=ON` when you build? Or is that the default on Linux now? If you're setting it explicitly, then you probably don't want to be doing that. `LLVM_ENABLE_PER_TARGET_RUNTIME_DIR=ON` is explicitly meant to have one `include/` and `lib/` dir for each target. If that's not your intent, then just use the good old way of doing things, i.e. just a single `include/c++/v1` directory with all the headers, including the `__config_site` header. If `LLVM_ENABLE_PER_TARGET_RUNTIME_DIR=ON` is the default on Linux, then perhaps you should explicitly specify `LLVM_ENABLE_PER_TARGET_RUNTIME_DIR=OFF`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D97572