mehdi_amini added a comment.

In https://reviews.llvm.org/D26649#595296, @phosek wrote:

> It's sufficient, I just tested it. I'm not actually sure if `LLVM_ENABLE_LLD` 
> here is correct, `LLVM_ENABLE_LLD` forces the use of lld, but lld might not 
> be available during first stage. We need `LLVM_ENABLE_LLD` to be set, but 
> only for the second stage (which is something that can be done in the cache 
> file), in the first stage we should probably just check whether lld source is 
> available and we're building it, but I'm not sure what's the easiest/cleanest 
> way to do that?


`-DBOOTSTRAP_ LLVM_ENABLE_LLD` is the right way to check for stage-2 options.

If you look just a few lines above your check you'll see an example of check 
for `BOOTSTRAP_LLVM_ENABLE_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