[Lldb-commits] [PATCH] D43591: [testsuite] Run lit tests as part of `check-lldb`

[Davide Italiano via Phabricator via lldb-commits]

davide created this revision.
davide added reviewers: friss, vsk, JDevlieghere, labath, zturner.
Herald added a subscriber: mgorny.

Also, fix a missing dependency, as lit requires `llvm-config` to run.
This is becoming more and more important as we write more FileCheck style tests (see Jonas' last fix to the expression parser)


https://reviews.llvm.org/D43591

Files:
  lldb/lit/CMakeLists.txt
  lldb/test/CMakeLists.txt


Index: lldb/test/CMakeLists.txt
===================================================================
--- lldb/test/CMakeLists.txt
+++ lldb/test/CMakeLists.txt
@@ -144,6 +144,9 @@
   endif ()
 endif ()

+# Run the lit-style tests and the unittests as part of the check-lldb target.
+add_dependencies(check-lldb check-lldb-lit)
+
 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.
Index: lldb/lit/CMakeLists.txt
===================================================================
--- lldb/lit/CMakeLists.txt
+++ lldb/lit/CMakeLists.txt
@@ -40,6 +40,7 @@
   LLDBUnitTests
   lldb
   lldb-test
+  llvm-config
   )

 if(NOT LLDB_BUILT_STANDALONE)


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D43591.135313.patch
Type: text/x-patch
Size: 782 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20180221/55773ec4/attachment.bin>