[PATCH] D121727: [NVPTX] Integrate ptxas to LIT tests

Joel E. Denny via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 18 15:14:20 PDT 2022


jdenny added a comment.

In D121727#3393481 <https://reviews.llvm.org/D121727#3393481>, @tra wrote:

> I believe that the whole "RUN" line must be handled atomically.

At this point, multiple use cases beyond the specific ptxas verification use case that started this thread are arising, so it sounds like a new construct that's general enough to handle them all is warranted.  For that general case, I agree with your arguments along the above lines.  Thanks for explaining your position.

> I can think of using the output of `ptxas -v` to catch at least one kind of regressions we've ran into in the past.

Thanks.  I wasn't aware of that use case.


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