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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to