[PATCH] D140370: [VP][RISCV] Add vp.ctlz/cttz and RISC-V support.

Yeting Kuo via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 21 17:21:20 PST 2022


fakepaper56 added inline comments.


================
Comment at: llvm/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp:7788
+  case ISD::VP_CTTZ_ZERO_UNDEF: {
+    OpValues.pop_back();
+    SDValue Result = DAG.getNode(Opcode, DL, VTs, OpValues);
----------------
Would it be better to skip Immarg argument in the below `Request operands` loop?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D140370



More information about the llvm-commits mailing list