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

[Zhihao Yuan via Phabricator via llvm-commits]

lichray added inline comments.


================
Comment at: cmake/modules/AddLLVM.cmake:153
+  if( LLVM_USE_LINKER )
+    set(command ${CMAKE_C_COMPILER} -fuse-ld=${LLVM_USE_LINKER} -Wl,--version)
+  else()
----------------
ruiu wrote:
> lichray wrote:
> > ruiu wrote:
> > > Is this safe if a C compiler does not support the -fuse-ld option?
> > That may be too old to compile LLVM.  I would say wait them come to us.  After all, this option is to assist detection, and more users are specifying linkers directly.
> You can use compilers other than clang to build clang and llvm, and I believe only clang supports this -fuse-ld option.
No?  gcc does https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55470 therefore Intel C++ does.  Clang does, then so does XL on AIX and Oracle.


Repository:
  rL LLVM

https://reviews.llvm.org/D38665