[llvm] r363015 - Setup testing target dependencies for default runtimes

Chris Bieneman via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 10 17:25:57 PDT 2019


Author: cbieneman
Date: Mon Jun 10 17:25:57 2019
New Revision: 363015

URL: http://llvm.org/viewvc/llvm-project?rev=363015&view=rev
Log:
Setup testing target dependencies for default runtimes

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

Modified:
    llvm/trunk/runtimes/CMakeLists.txt

Modified: llvm/trunk/runtimes/CMakeLists.txt
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/runtimes/CMakeLists.txt?rev=363015&r1=363014&r2=363015&view=diff
==============================================================================
--- llvm/trunk/runtimes/CMakeLists.txt (original)
+++ llvm/trunk/runtimes/CMakeLists.txt Mon Jun 10 17:25:57 2019
@@ -352,6 +352,7 @@ else() # if this is included from LLVM's
     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
       runtime_default_target(
         DEPENDS ${deps}
         PREFIXES ${prefixes})
+      set(test_targets check-runtimes)
     else()
       if("default" IN_LIST LLVM_RUNTIME_TARGETS)
         runtime_default_target(




More information about the llvm-commits mailing list