This is an archive of the discontinued LLVM Phabricator instance.

Fix running orc-rt tests with LLVM_BUILD_EXTERNAL_COMPILER_RT
ClosedPublic

Authored by benlangmuir on Jan 24 2022, 4:09 PM.

Details

Summary

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 Timeline

benlangmuir created this revision.Jan 24 2022, 4:09 PM
benlangmuir requested review of this revision.Jan 24 2022, 4:09 PM
Herald added a project: Restricted Project. · View Herald TranscriptJan 24 2022, 4:09 PM
Herald added a subscriber: cfe-commits. · View Herald Transcript
lhames accepted this revision.Jan 24 2022, 4:21 PM

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?

This revision is now accepted and ready to land.Jan 24 2022, 4:21 PM

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?

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
This revision was landed with ongoing or failed builds.Jan 25 2022, 8:28 AM
This revision was automatically updated to reflect the committed changes.