Skip to content

Commit

Permalink
[Runtimes] If LLVM_INCLUDE_TESTS=On depend on gtest
Browse files Browse the repository at this point in the history
Summary: 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.

Reviewers: phosek, compnerd

Reviewed By: compnerd

Subscribers: mgorny, winksaville, llvm-commits

Tags: #llvm

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

llvm-svn: 361436
  • Loading branch information
llvm-beanz committed May 22, 2019
1 parent e3748b5 commit ed00367
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions llvm/runtimes/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -553,6 +553,8 @@ else() # if this is included from LLVM's CMake
obj2yaml
sancov
sanstats
gtest_main
gtest
)
foreach(target ${test_targets} ${SUB_CHECK_TARGETS})
add_dependencies(${target} ${RUNTIMES_TEST_DEPENDS})
Expand Down

0 comments on commit ed00367

Please sign in to comment.