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

[Chris Bieneman via Phabricator via llvm-commits]

beanz added inline comments.


================
Comment at: runtimes/CMakeLists.txt:429
+      add_dependencies(clang-bootstrap-deps runtimes)
       add_dependencies(clang-bootstrap-deps runtimes-configure)
     endif()
----------------
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...).


https://reviews.llvm.org/D35648