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

Andrew Savonichev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 20 12:49:31 PDT 2022


asavonic added a comment.

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

> In D121727#3462870 <https://reviews.llvm.org/D121727#3462870>, @asavonic wrote:
>
>> One disadvantage of this patch is that we can no longer pipe llc output to FileCheck: we have to use `-o` option with a temporary file. This may break update_llc_checks.py, but I haven't confirmed this yet.
>
> We don't have to use a temporary file, I think. ptxas can get its own input from its own llc invocation
> ` echo -e '.version 3.2\n.target sm_20\n.address_size 64' | $HOME/local/cuda-11.2/bin/ptxas -` works for me.

Right, but then we can't pipe it to file check. Something like this should work, but it is not portable:

  llc ... | tee %t.ptx | FileCheck
  ptxas %.ptx


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