[PATCH] D51804: [CMake] Fix LTO option on Windows

[Zachary Turner via llvm-commits]

Is this a bug in CMake?  Maybe you should report it on their bug tracker.

On Fri, Sep 7, 2018 at 1:08 PM Alexandre Ganea via Phabricator <
reviews at reviews.llvm.org> wrote:

> This revision was automatically updated to reflect the committed changes.
> Closed by commit rL341701: [CMake] Fix LLVM_ENABLE_LTO option on Windows
> (authored by aganea, committed by ).
>
> Changed prior to commit:
>   https://reviews.llvm.org/D51804?vs=164458&id=164495#toc
>
> Repository:
>   rL LLVM
>
> https://reviews.llvm.org/D51804
>
> Files:
>   llvm/trunk/cmake/modules/HandleLLVMOptions.cmake
>
>
> Index: llvm/trunk/cmake/modules/HandleLLVMOptions.cmake
> ===================================================================
> --- llvm/trunk/cmake/modules/HandleLLVMOptions.cmake
> +++ llvm/trunk/cmake/modules/HandleLLVMOptions.cmake
> @@ -11,7 +11,7 @@
>  include(CheckCCompilerFlag)
>  include(CheckCXXCompilerFlag)
>
> -if(CMAKE_LINKER MATCHES "lld-link.exe" OR (WIN32 AND LLVM_USE_LINKER
> STREQUAL "lld"))
> +if(CMAKE_LINKER MATCHES "lld-link\.exe" OR (WIN32 AND LLVM_USE_LINKER
> STREQUAL "lld") OR LLVM_ENABLE_LLD)
>    set(LINKER_IS_LLD_LINK TRUE)
>  else()
>    set(LINKER_IS_LLD_LINK FALSE)
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180907/e25994b7/attachment.html>