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

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 2 10:46:41 PST 2022


craig.topper added inline comments.


================
Comment at: llvm/lib/Target/RISCV/RISCVISelDAGToDAG.cpp:2196
+
+  if (RISCVII::hasVecPolicyOp(MaskedMCID.TSFlags)) {
+    // The last operand of the pseudo is the policy op, but we're expecting a
----------------
If the policy Op exists and isn't TAIL_AGNOSTIC, this transform isn't valid is it?

Using an all ones mask with TAIL_UNDISTURBED is a valid way to get tail undisturbed behavior until we have tail undisturbed unmasked instructions.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D118810



More information about the llvm-commits mailing list