[PATCH] D133869: [RISCV] Verify SEW/VecPolicy immediate values

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 14 12:52:05 PDT 2022


craig.topper added inline comments.


================
Comment at: llvm/lib/Target/RISCV/RISCVInstrInfo.cpp:1225
+  if (RISCVII::hasSEWOp(TSFlags)) {
+    unsigned OpIdx = RISCVII::getSEWOpNum(MI.getDesc());
+    unsigned Log2SEW = MI.getOperand(OpIdx).getImm();
----------------
MI.getDesc() here should be the same as `Desc` 2 lines up. But it looks like we took the long way to create `Desc` at the top of this function.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D133869



More information about the llvm-commits mailing list