[PATCH] D35648: [CMake] Add more runtime configurations.

[Leo Li via Phabricator via llvm-commits]

aoli added inline comments.


================
Comment at: runtimes/CMakeLists.txt:429
+      add_dependencies(clang-bootstrap-deps runtimes)
       add_dependencies(clang-bootstrap-deps runtimes-configure)
     endif()
----------------
beanz wrote:
> srhines wrote:
> > Is this dependency needed now? I assume runtimes depends on runtimes-configure, but maybe I am mistaken.
> Uh... This part of the patch should be done differently.
> 
> This hack is in place because of how clang finds c++ headers relative to itself, so we need to configure the runtimes directory (which copies the c++ headers) otherwise we can't actually build anything with the stage1 compiler.
> 
> Making clang-bootstrap-deps depend on the runtimes target should be done separately only if the next stage build is intending to link against runtime libraries from the stage1 build, and even then I'm not sure we should be depending on the runtimes target, I'd much rather the dependency was setup on the top-level target for the runtime library (i.e. libcxx, libunwind, compiler-rt...).
I agree. 

Shall we bring some new variables like `LLVM_USE_STAGE1_LIBCXX`, `LLVM_USE_STAGE1_LIBUNWIND` ...?


https://reviews.llvm.org/D35648