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

[Louis Dionne via Phabricator via libcxx-commits]

ldionne added a comment.

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`?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D82702