benlangmuir added a comment. > 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 Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D118087/new/ https://reviews.llvm.org/D118087 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits