[PATCH] D26649: [CMake] Support lld with LTO bootstrap

[Chris Bieneman via Phabricator via cfe-commits]

beanz added inline comments.


================
Comment at: cfe/trunk/CMakeLists.txt:531
+      if(BOOTSTRAP_LLVM_ENABLE_LLD)
+        add_dependencies(clang-bootstrap-deps lld)
+      elseif(LLVM_BINUTILS_INCDIR)
----------------
mehdi_amini wrote:
> I come back to this a bit late, sorry, but I'm not sure I understand why this dependency on the bootstrap happens only when LTO is used?
> 
We should probably separate `LLVM_ENABLE_LTO` and `LLVM_ENABLE_LLD`. You can use LLD without LTO on a wider variety of platforms (including OS X), and you might want to use LLD without LTO.


Repository:
  rL LLVM

https://reviews.llvm.org/D26649