From: Marcos Pividori Date: Fri, 20 Jan 2017 22:49:08 +0000 (+0000) Subject: [libFuzzer] Use clang as linker on Windows, to properly include sanitizer libraries. X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=3984eb4ad0efd3ed9536a295f57d4cea88a854b4;p=platform%2Fupstream%2Fllvm.git [libFuzzer] Use clang as linker on Windows, to properly include sanitizer libraries. In order to use sanitizers on Windows, we need to link against many runtime libraries which will depend on the target being created (executable or dll) and the c runtime library used (MT/MD). By default, cmake uses link.exe for linking, which fails because we don't specify the appropiate dependencies. As we don't want to consider all of that possible situations which depends on the implementation of the compiler-rt, the simplest option is to change the rules for linking executables and shared libraries, using the compiler instead of link.exe. Clang driver will consider the sanitizer flags, and automatically provide the required libraries to the linker. Differential Revision: https://reviews.llvm.org/D27869 llvm-svn: 292669 --- diff --git a/llvm/lib/Fuzzer/test/CMakeLists.txt b/llvm/lib/Fuzzer/test/CMakeLists.txt index ed84313..18e9e81 100644 --- a/llvm/lib/Fuzzer/test/CMakeLists.txt +++ b/llvm/lib/Fuzzer/test/CMakeLists.txt @@ -27,6 +27,21 @@ endforeach() # Enable the coverage instrumentation (it is disabled for the Fuzzer lib). set(CMAKE_CXX_FLAGS "${LIBFUZZER_FLAGS_BASE} -fsanitize-coverage=trace-pc-guard,indirect-calls,trace-cmp,trace-div,trace-gep -gline-tables-only") +if(MSVC) + # In order to use the sanitizers in Windows, we need to link against many + # runtime libraries which will depend on the target being created + # (executable or dll) and the c runtime library used (MT/MD). + # By default, cmake uses link.exe for linking, which fails because we don't + # specify the appropiate dependencies. + # As we don't want to consider all of that possible situations which depends + # on the implementation of the compiler-rt, the simplest option is to change + # the rules for linking executables and shared libraries, using the compiler + # instead of link.exe. Clang will consider the sanitizer flags, and + # automatically provide the required libraries to the linker. + set(CMAKE_CXX_LINK_EXECUTABLE " ${CMAKE_CXX_FLAGS} -o /link ") + set(CMAKE_CXX_CREATE_SHARED_LIBRARY " ${CMAKE_CXX_FLAGS} /LD -o /link ") +endif() + # add_libfuzzer_test( # SOURCES source0.cpp [source1.cpp ...] # )