[PATCH] D22896: [LLVM] Add LLVM_ENABLE_LLD option to use LLD as C/C++ linker.

[Eugene Zelenko via llvm-commits]

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