[ORC-RT] Don't unconditionally add dependence on llvm-jitlink.
authorLang Hames <lhames@gmail.com>
Fri, 2 Sep 2022 05:42:39 +0000 (22:42 -0700)
committerLang Hames <lhames@gmail.com>
Fri, 2 Sep 2022 05:53:30 +0000 (22:53 -0700)
Commit 4adc5bead4a moved a dependence on llvm-jitlink from
SANITIZER_COMMON_LIT_TEST_DEPS to ORC_TEST_DEPS, but in doing so it moved it
out from under a 'NOT COMPILER_RT_STANDALONE_BUILD ...' conditional. This led
to failures on standalone builds.

This commit adds the conditional to the ORC_TEST_DEPS assignment to work
around the issue while we look a longer term fix.

rdar://99453446

compiler-rt/test/orc/CMakeLists.txt

index 36a7462..4d3e80c 100644 (file)
@@ -1,8 +1,16 @@
 set(ORC_LIT_SOURCE_DIR ${CMAKE_CURRENT_SOURCE_DIR})
 
 set(ORC_TESTSUITES)
-set(ORC_TEST_DEPS)
-list(APPEND ORC_TEST_DEPS ${SANITIZER_COMMON_LIT_TEST_DEPS} llvm-jitlink)
+set(ORC_TEST_DEPS ${SANITIZER_COMMON_LIT_TEST_DEPS})
+
+# TODO: This conditional was added to get standalone builds working
+#       (rdar://99453446). Longer term we should aim to identify and disable
+#       tests that depend on llvm-jitlink, or perhaps just disable ORC runtime
+#       regression testing entirely when it's not available.
+if (NOT ANDROID AND NOT COMPILER_RT_STANDALONE_BUILD AND
+    NOT LLVM_RUNTIMES_BUILD)
+  list(APPEND ORC_TEST_DEPS $llvm-jitlink)
+endif()
 
 set(ORC_TEST_ARCH ${ORC_SUPPORTED_ARCH})
 if (COMPILER_RT_BUILD_ORC AND COMPILER_RT_HAS_ORC)