[PATCH] D27993: [libFuzzer] Diff 44 - Remove dependency for tests on Windows.

[Marcos Pividori via Phabricator via llvm-commits]

This revision was automatically updated to reflect the committed changes.
Closed by commit rL292735: [libFuzzer] Remove dependencies for tests on Windows. (authored by mpividori).

Changed prior to commit:
  https://reviews.llvm.org/D27993?vs=83180&id=85256#toc

Repository:
  rL LLVM

https://reviews.llvm.org/D27993

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


Index: llvm/trunk/lib/Fuzzer/test/CMakeLists.txt
===================================================================
--- llvm/trunk/lib/Fuzzer/test/CMakeLists.txt
+++ llvm/trunk/lib/Fuzzer/test/CMakeLists.txt
@@ -231,5 +231,10 @@
 
 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()


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D27993.85256.patch
Type: text/x-patch
Size: 615 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170122/d259bc66/attachment.bin>