[PATCH] D141460: [bootstrap build] Add libc-hdrgen as a build dep for libc bootstrap build.

[Siva Chandra via Phabricator via llvm-commits]

sivachandra added inline comments.


================
Comment at: llvm/runtimes/CMakeLists.txt:391-392
+    else()
+      # We want to build the libc build tools before we can build the libc
+      # itself. So, the libc project should be included in LLVM_ENABLE_PROJECTS.
+      message(FATAL_ERROR
----------------
phosek wrote:
> This is slightly counterintuitive. I'd prefer introducing a special handling in `llvm/CMakeLists.txt` similarly to what we do for example for `flang` and `mlir`, see https://github.com/llvm/llvm-project/blob/830d116d776439093a16aad567cd3db2c4c3cd72/llvm/CMakeLists.txt#L143 and automatically including `libc` in `LLVM_ENABLE_PROJECTS` when `libc` is in `LLVM_ENABLE_RUNTIMES`.
I left this check here as a debug aid, but have also updated `llvm/CMakeLists.txt` according to your suggestion.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D141460