From 4eb84582344f97167b6a2b4cb1fb1d75ae07897e Mon Sep 17 00:00:00 2001 From: Shoaib Meenai Date: Wed, 30 Nov 2022 16:41:31 -0800 Subject: [runtimes] Fix runtimes-test-depends The dependency list is stored in a global property, so we need to fetch it to a variable before using that variable. We also need to add the list contents as dependencies correctly. --- runtimes/CMakeLists.txt | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) (limited to 'runtimes') diff --git a/runtimes/CMakeLists.txt b/runtimes/CMakeLists.txt index df279bb..39c58f9 100644 --- a/runtimes/CMakeLists.txt +++ b/runtimes/CMakeLists.txt @@ -232,7 +232,12 @@ if(LLVM_INCLUDE_TESTS) # LLVM_RUNTIMES_LIT_DEPENDS is populated when lit tests are added between # umbrella_list_testsuite begin and end. The bootstrap runtimes builds # currently assumes this target exists. - add_custom_target(runtimes-test-depends ${LLVM_RUNTIMES_LIT_DEPENDS}) + get_property(LLVM_RUNTIMES_LIT_DEPENDS GLOBAL PROPERTY LLVM_RUNTIMES_LIT_DEPENDS) + add_custom_target(runtimes-test-depends) + if(LLVM_RUNTIMES_LIT_DEPENDS) + # add_dependencies complains if called with no dependencies + add_dependencies(runtimes-test-depends ${LLVM_RUNTIMES_LIT_DEPENDS}) + endif() # Add a global check rule now that all subdirectories have been traversed # and we know the total set of lit testsuites. umbrella_lit_testsuite_end(check-runtimes) -- cgit v1.1