[PATCH] D62269: [Runtimes] If LLVM_INCLUDE_TESTS=On depend on gtest
Chris Bieneman via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed May 22 13:43:35 PDT 2019
beanz created this revision.
beanz added reviewers: phosek, compnerd.
Herald added a subscriber: mgorny.
Herald added a project: LLVM.
If we are building the tests for the runtimes we should make them depend on gtest so that gtest is built and ready before we run any of the check-* targets.
Repository:
rG LLVM Github Monorepo
https://reviews.llvm.org/D62269
Files:
llvm/runtimes/CMakeLists.txt
Index: llvm/runtimes/CMakeLists.txt
===================================================================
--- llvm/runtimes/CMakeLists.txt
+++ llvm/runtimes/CMakeLists.txt
@@ -303,6 +303,10 @@
endif()
endif()
+ if(LLVM_INCLUDE_TESTS)
+ list(APPEND deps gtest_main gtest)
+ endif()
+
# We create a list the names of all the runtime projects in all uppercase and
# with dashes turned to underscores. This gives us the CMake variable prefixes
# for all variables that will apply to runtimes.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D62269.200816.patch
Type: text/x-patch
Size: 511 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190522/7f8194c5/attachment.bin>
More information about the llvm-commits
mailing list