[PATCH] D25263: [Driver] Allow setting the default linker during build

[Bruno Cardoso Lopes via cfe-commits]

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