abrachet added a comment.
In D128567#3863178 <https://reviews.llvm.org/D128567#3863178>, @aeubanks wrote:
> I've noticed that we're skipping compiler-rt tests with `REQUIRED:
> lld-available` because `LLVM_TOOL_LLD_BUILD` needs to be defined. any update
> on the proposed alternative?
I think the canonical way this is done throughout the code base is with
from lit.llvm import llvm_config
if llvm_config.use_lld(required=False):
config.available_features.add('lld')
Though, what I think we don't do currently here or elsewhere is conditionally
add test deps on `lld` if it is specified in `LLVM_ENABLE_PROJECTS`
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D128567/new/
https://reviews.llvm.org/D128567
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
