[Lldb-commits] [PATCH] D39689: Add a dependency from check-lldb on lld

[Stephane Sezer via Phabricator via lldb-commits]

sas updated this revision to Diff 121760.
sas added a comment.

Move check out of if (TARGET clang) block.


https://reviews.llvm.org/D39689

Files:
  test/CMakeLists.txt


Index: test/CMakeLists.txt
===================================================================
--- test/CMakeLists.txt
+++ test/CMakeLists.txt
@@ -115,6 +115,15 @@
   add_dependencies(check-lldb clang)
 endif()
 
+# LLD is required to link test executables on Windows.
+if (CMAKE_SYSTEM_NAME MATCHES "Windows")
+  if (TARGET lld)
+    add_dependencies(check-lldb lld)
+  else ()
+    message(WARNING "lld required to test LLDB on Windows")
+  endif ()
+endif ()
+
 add_custom_target(lldb-test-depends DEPENDS ${LLDB_TEST_DEPENDS})
 # This will add LLDB's test dependencies to the depenednecies for check-all and
 # include them in the test-depends target.


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D39689.121760.patch
Type: text/x-patch
Size: 656 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20171106/f8fa08af/attachment.bin>