[PATCH] D99158: [RISCV][WIP] Implement intrinsics for P extension

[Jim Lin via Phabricator via llvm-commits]

Jim added inline comments.


================
Comment at: llvm/lib/Target/RISCV/RISCVISelLowering.cpp:706
+
+      setOperationAction(ISD::BITCAST, VT, Legal);
+
----------------
Jim wrote:
> craig.topper wrote:
> > What about bitcast from float/double to any of these vector types? I'm guess that's not legal.
> Yes, it is not legal bitcast from float/double to any of these vector types.
> All operations of these vector types have been expanded on line 760.
> Only bitcast from/to v4i8/v2i16, v8i8/v4i16/v2i32 to/from i32, i64 are legal.
Oh, it is a issue. I will try to fix it.


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

https://reviews.llvm.org/D99158