[PATCH] D121727: [NVPTX] Integrate ptxas to LIT tests
Nico Weber via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Apr 28 05:46:06 PDT 2022
thakis added a comment.
It looks very weird to me to have LLVM's test suite support calling some proprietary tool. I can see how this is useful, but imho it doesn't belong into LLVM itself.
This is both for practical and other reasons.
>From a practical point of view, if we rely on this tool for testing in LLVM, it means we can't run LLVM's tests on platforms that that tool doesn't run on.
Independent of practical considerations, LLVM tries to be an open-source, standalone toolchain development suite. Making it depend on proprietary programs, even just optionally and at the test level, works against both these goals.
How do others feel about this?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D121727/new/
https://reviews.llvm.org/D121727
More information about the llvm-commits
mailing list