[PATCH] D51804: [CMake] Fix LTO option on Windows
Alexandre Ganea via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Oct 5 20:22:35 PDT 2018
aganea added a comment.
In https://reviews.llvm.org/D51804#1257130, @azharudd wrote:
> > CMAKE_LINKER MATCHES "lld-link\.exe"
>
> @aganea, can you please elaborate on the need for the backslash that you added?
It is not needed //per se//. But given than MATCHES takes a regular expression on the right side, I machinally replaced the . (maching any character) by \. (escaped dot) while looking for a bug. Is that an issue?
Repository:
rL LLVM
https://reviews.llvm.org/D51804
More information about the llvm-commits
mailing list