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

Wed Jul 15 09:40:28 PDT 2020

thakis added a comment.

In D82702#2153130 <https://reviews.llvm.org/D82702#2153130>, @ldionne wrote:

> In D82702#2151903 <https://reviews.llvm.org/D82702#2151903>, @aeubanks wrote:
>
> > Is it ok to revert this then?
>
>
> No.
>
> The issue is that your builder builds with `-DLIBCXX_ENABLE_SHARED=OFF -DLIBCXX_ENABLE_STATIC=OFF`, but `LIBCXX_ENABLE_EXPERIMENTAL_LIBRARY` is implicitly set to `ON`. In other words, you're trying to build the libc++ experimental library without building the static or the shared library. If you're trying to disable libc++ from the build, why include it in `LLVM_ENABLE_PROJECTS`?

See https://bugs.chromium.org/p/chromium/issues/detail?id=1067216#c7 for background :)

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D82702