[PATCH] D121881: [RISCV][RVV] Select unmasked TU RVV pseudos in a DAG post-process

ShihPo Hung via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 8 06:46:19 PDT 2022


arcbbb added inline comments.


================
Comment at: llvm/lib/Target/RISCV/RISCVInstrInfoVPseudos.td:429
   Pseudo UnmaskedPseudo = !cast<Pseudo>(!subst("_MASK", "", NAME));
+  Pseudo UnmaskedTuPseudo = !cast<Pseudo>(!subst("_MASK", "", NAME # "_TU"));
   bits<4> MaskOpIdx = MaskIdx;
----------------
frasercrmck wrote:
> arcbbb wrote:
> > After D123217 there are cases where _TU pseudo doesn't exist.
> > I have to fix this logic.
> I'm sorry about that. Can we make it optional somehow?
It just happened sooner than I thought it would be.
Not sure if I have done it right, I create a pseudo as a placeholder.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D121881/new/

https://reviews.llvm.org/D121881



More information about the llvm-commits mailing list