[PATCH] D94766: Add lit env variable to disable indirect checks
Joel E. Denny via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Jan 21 10:31:00 PST 2021
jdenny added a comment.
So the lit invocations are in a script of some sort, right? Such a script could perform a feature test to handle this use case: if `--no-indirectly-run-check` is supported, add it to some lit options variable used by all relevant lit invocations in the script, and do not otherwise. Is that difficult to do?
I'm not strongly opposed to this patch, especially if it helps many people. However, given that this use case is external and temporary, I'm inclined to think a feature test is more reasonable: it's also simple, but it's external like the use case and can be dropped by users when they decide they no longer need to support old lit versions. What do you think?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D94766/new/
https://reviews.llvm.org/D94766
More information about the llvm-commits
mailing list