[PATCH] D142102: [RISCV] Combine FP_TO_INT to vfwcvt/fvncvt
Luke Lau via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Jan 23 10:15:12 PST 2023
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
More information about the llvm-commits
mailing list