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
Differential D118087
Fix running orc-rt tests with LLVM_BUILD_EXTERNAL_COMPILER_RT benlangmuir on Jan 24 2022, 4:09 PM. Authored by
Details 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
Diff Detail
Event TimelineComment Actions LGTM! Does check-orc-rt need to be added to COMPILER_RT_TEST_SUITES too? I guess not, otherwise we wouldn't have seen this missing dependency in the first place? Comment Actions
It doesn't affect check-compiler-rt, because check-compiler-rt actually wraps the check-all target from compiler-rt rather than depending on the individual components defined in this file. I think we only need to add individual components here if it's useful to someone. There's actually a whole bunch of check- targets in compiler-rt that are not reflected here, so I'd prefer to call it a separate issue if we do add more: check-asan-${platform}-${arch} check-builtins check-crt check-fuzzer-${platform}-${arch} check-fuzzer-${stdlib_name}-${arch} check-fuzzer-unit check-hwasan-lam check-interception check-memprof check-memprof-dynamic check-orc-rt check-scudo check-scudo_standalone check-shadowcallstack check-tsan-${platform}-${arch} check-tsan-dynamic check-ubsan-${test_mode}-${platform}-${arch} check-xray |