[PATCH] D22896: [LLVM] Add LLVM_ENABLE_LLD option to use LLD as C/C++ linker.
Eugene Zelenko via llvm-commits
llvm-commits at lists.llvm.org
Wed Jul 27 18:59:43 PDT 2016
Eugene.Zelenko added inline comments.
================
Comment at: cmake/modules/HandleLLVMOptions.cmake:15
@@ -14,3 +14,2 @@
-
if (CMAKE_LINKER MATCHES "lld-link.exe")
----------------
compnerd wrote:
> Why the whitespace removal?
Is it really necessary? I don't see any logic behind. Same for white space below.
================
Comment at: cmake/modules/HandleLLVMOptions.cmake:15
@@ -14,3 +14,2 @@
-
if (CMAKE_LINKER MATCHES "lld-link.exe")
----------------
Eugene.Zelenko wrote:
> compnerd wrote:
> > Why the whitespace removal?
> Is it really necessary? I don't see any logic behind. Same for white space below.
Is it really necessary? I don't see any logic behind. Same for white space below.
Repository:
rL LLVM
https://reviews.llvm.org/D22896
More information about the llvm-commits
mailing list