[PATCH] D122860: [RISCV][RVV] Add FPRndModeOp to PseudoVFCVT instructions

Yeting Kuo via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 7 18:43:53 PDT 2022


fakepaper56 added inline comments.
Herald added a subscriber: shiva0217.


================
Comment at: llvm/lib/Target/RISCV/RISCVISelLowering.cpp:9543
+    (void)HasSEW;
+    assert(HasVL && HasSEW);
+    Idx = MI.getNumExplicitOperands() - HasPolicy - 3;
----------------
How about just call `hasVLOp` and `hasSEWOp` to avoid the two casts to `void`?
```
assert(RISCVII::hasVLOp(TSFlags) && RISCVII::hasSEWOp(TSFlags));
```


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D122860



More information about the llvm-commits mailing list