[PATCH] D142102: [RISCV] Combine FP_TO_INT to vfwcvt/fvncvt

[Luke Lau via Phabricator via llvm-commits]

luke added inline comments.


================
Comment at: llvm/lib/Target/RISCV/RISCVISelLowering.h:240
   VFCVT_RTZ_XU_F_VL,
-  VFCVT_X_F_VL,
+  VFCVT_RTZ_X_F_VL,
   VFCVT_XU_F_VL,
----------------
craig.topper wrote:
> Why changing the order? This puts in a different order than the list in RISCVTargetLowering::getTargetNodeName
I was trying to order it to match the spec, since the unsigned instructions seem to come before the signed instructions. Will take this out of this patch anyway


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D142102