phosek 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) ---------------- beanz wrote: > 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. Yes, this is unnecessarily restrictive; see D28821. 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
