[llvm] r292735 - [libFuzzer] Remove dependencies for tests on Windows.

Marcos Pividori via llvm-commits llvm-commits at lists.llvm.org
Sat Jan 21 17:27:34 PST 2017


Author: mpividori
Date: Sat Jan 21 19:27:34 2017
New Revision: 292735

URL: http://llvm.org/viewvc/llvm-project?rev=292735&view=rev
Log:
[libFuzzer] Remove dependencies for tests on Windows.

Remove dependency on FileCheck, sancov and not for tests on Windows.
If LLVM_USE_SANITIZER=Address and LLVM_USE_SANITIZE_COVERAGE=YES, this will
trigger the building of dependencies with sanitizer instrumentation.
This will fail in Windows, since cmake will use link.exe for linking and won't
include compiler-rt libraries.

Differential Revision: https://reviews.llvm.org/D27993

Modified:
    llvm/trunk/lib/Fuzzer/test/CMakeLists.txt

Modified: llvm/trunk/lib/Fuzzer/test/CMakeLists.txt
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Fuzzer/test/CMakeLists.txt?rev=292735&r1=292734&r2=292735&view=diff
==============================================================================
--- llvm/trunk/lib/Fuzzer/test/CMakeLists.txt (original)
+++ llvm/trunk/lib/Fuzzer/test/CMakeLists.txt Sat Jan 21 19:27:34 2017
@@ -231,5 +231,10 @@ configure_lit_site_cfg(
 
 add_lit_testsuite(check-fuzzer "Running Fuzzer tests"
     ${CMAKE_CURRENT_BINARY_DIR}
-    DEPENDS ${TestBinaries} FileCheck sancov not
     )
+
+# Don't add dependencies on Windows. The linker step would fail on Windows,
+# since cmake will use link.exe for linking and won't include compiler-rt libs.
+if(NOT MSVC)
+  add_dependencies(check-fuzzer FileCheck sancov not)
+endif()




More information about the llvm-commits mailing list