Skip to content

Commit

Permalink
Setup testing target dependencies for default runtimes
Browse files Browse the repository at this point in the history
Summary: The default runtimes targets aren't getting their dependencies configured correctly which results in check-runtimes failing when built from a clean build.

Reviewers: phosek, compnerd

Reviewed By: phosek

Subscribers: mgorny, llvm-commits

Tags: #llvm

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

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@363015 91177308-0d34-0410-b5e6-96231b3b80d8
llvm-beanz committed Jun 11, 2019
1 parent 0a0d6b7 commit ef35a12
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions runtimes/CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -352,6 +352,7 @@ else() # if this is included from LLVM's CMake
cmake_parse_arguments(ARG "" "" "DEPENDS;PREFIXES" ${ARGN})

include(${LLVM_BINARY_DIR}/runtimes/Components.cmake OPTIONAL)
set(SUB_CHECK_TARGETS ${SUB_CHECK_TARGETS} PARENT_SCOPE)
set_property(DIRECTORY APPEND PROPERTY CMAKE_CONFIGURE_DEPENDS ${LLVM_BINARY_DIR}/runtimes/Components.cmake)

foreach(runtime_name ${runtime_names})
@@ -497,6 +498,7 @@ else() # if this is included from LLVM's CMake
runtime_default_target(
DEPENDS ${deps}
PREFIXES ${prefixes})
set(test_targets check-runtimes)
else()
if("default" IN_LIST LLVM_RUNTIME_TARGETS)
runtime_default_target(

0 comments on commit ef35a12

Please sign in to comment.