benlangmuir created this revision.
benlangmuir added reviewers: lhames, fhahn.
Herald added a subscriber: mgorny.
benlangmuir requested review of this revision.
Herald added a project: clang.
Herald added a subscriber: cfe-commits.
Add missing dependency on llvm-jitlink when building compiler-rt with
LLVM_BUILD_EXTERNAL_COMPILER_RT. Previously we would non-deterministically fail
the tests due to the missing binary.
rdar://87247681
Repository:
rG LLVM Github Monorepo
https://reviews.llvm.org/D118087
Files:
clang/runtime/CMakeLists.txt
Index: clang/runtime/CMakeLists.txt
===================================================================
--- clang/runtime/CMakeLists.txt
+++ clang/runtime/CMakeLists.txt
@@ -130,7 +130,7 @@
if(LLVM_INCLUDE_TESTS)
# Add binaries that compiler-rt tests depend on.
set(COMPILER_RT_TEST_DEPENDENCIES
- FileCheck count not llvm-nm llvm-objdump llvm-symbolizer)
+ FileCheck count not llvm-nm llvm-objdump llvm-symbolizer llvm-jitlink)
# Add top-level targets for various compiler-rt test suites.
set(COMPILER_RT_TEST_SUITES check-fuzzer check-asan check-hwasan
check-asan-dynamic check-dfsan
Index: clang/runtime/CMakeLists.txt
===================================================================
--- clang/runtime/CMakeLists.txt
+++ clang/runtime/CMakeLists.txt
@@ -130,7 +130,7 @@
if(LLVM_INCLUDE_TESTS)
# Add binaries that compiler-rt tests depend on.
set(COMPILER_RT_TEST_DEPENDENCIES
- FileCheck count not llvm-nm llvm-objdump llvm-symbolizer)
+ FileCheck count not llvm-nm llvm-objdump llvm-symbolizer llvm-jitlink)
# Add top-level targets for various compiler-rt test suites.
set(COMPILER_RT_TEST_SUITES check-fuzzer check-asan check-hwasan check-asan-dynamic check-dfsan
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
