[PATCH] D97301: [RISCV] Support fixed-length vector FP_ROUND & FP_EXTEND

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 23 09:59:40 PST 2021


craig.topper added inline comments.


================
Comment at: llvm/lib/Target/RISCV/RISCVISelLowering.h:181
+  // FP_ROUND_VL carries the extra TRUNC operand before the Mask/VL to maintain
+  // parity with the regular ISD node.
+  FP_ROUND_VL,
----------------
I doubt we need that TRUNC operand. I think its use in the regular node is for scalar call arguments.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D97301



More information about the llvm-commits mailing list