[PATCH] D121727: [NVPTX] Integrate ptxas to LIT tests
Artem Belevich via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Mar 15 12:12:44 PDT 2022
tra added a comment.
Substituting `true` may not always work. It's OK if we only need pass/fail result from ptxas, but we may want to check ptxas output, too.
Can we make `ptxas` a feature instead? The tests that need it would use `REQUIRES: ptxas` and would be guaranteed to have ptxas available.
It may also need to be versioned, too. Some tests may require recent enough version of ptxas in order to work (e.g. we may want to use it to test new instructions that an older ptxas may not understand).
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