[PATCH] D132923: [RISCV] Support peephole optimization to fold vmerge.vvm that has tail agnostic policy and unmasked intrinsics.

Jianjian Guan via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Sep 4 19:40:22 PDT 2022


jacquesguan added inline comments.


================
Comment at: llvm/lib/Target/RISCV/RISCVISelDAGToDAG.cpp:2715
     SmallVector<SDValue, 8> Ops;
-    Ops.push_back(Merge);
+    if (RISCVII::hasVecPolicyOp(TII->get(MaskedOpc).TSFlags)) 
+      Ops.push_back(False);
----------------
fakepaper56 wrote:
> Why you test vector policy of `MaskedOpc` here?
Thanks, it's a mistake, I am intended to check whether `MaskedOpc` has mergeop here. Fixed now.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D132923



More information about the llvm-commits mailing list