bruno added inline comments.

================
Comment at: CMakeLists.txt:198
 
+set(CLANG_DEFAULT_LINKER "" CACHE STRING
+  "Default linker to use (\"bfd\" or \"gold\" or \"lld\", empty for platform 
default")
----------------
mgorny wrote:
> Is there a reason not to allow using the absolute path here, like for the 
> command-line option?
I agree here, if we're adding a cmake options for this, it should accept full 
paths to the linker to be used (without any need for its type like gold, bfd, 
etc) as well.

Additionally, if "" maps to "ld", plain CLANG_DEFAULT_LINKER="ld" should also 
work here.


Repository:
  rL LLVM

https://reviews.llvm.org/D25263



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

Reply via email to