[PATCH] D122512: [RISCV] Add lowering for vp.fptosi and vp.sitofp.

[Craig Topper via Phabricator via llvm-commits]

craig.topper added inline comments.


================
Comment at: llvm/lib/Target/RISCV/RISCVISelLowering.cpp:6175
+  SDValue Result;
+  if (DstTypeSize >= SrcTypeSize) { // Single-width and widening conversion.
+    if (SrcType.isInteger()) {
----------------
Source repo had a separate else for DstTypeSize == SrcTypeSize. I merged them since the only difference was that else didn't check the pre-widening conditions. But the pre-widening conditions won't match for the single-width case so we can share the code.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D122512