[PATCH] D38665: [CMake] Fix linker detection in AddLLVM.cmake

Rui Ueyama via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 24 17:25:41 PDT 2017


ruiu added a comment.

I mean -DLLVM_USE_LINKER=lld is a valid option, so when it failed due to the lack compiler support of -fuse-ld=lld, it should be reported as a meaningful error, instead of a cryptic "CMake failed to compile this program blah blah." message. I think that is also true to the case when you pass -DLLVM_USE_LINKER=gold but gold was not installed to your system.


Repository:
  rL LLVM

https://reviews.llvm.org/D38665





More information about the llvm-commits mailing list