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

[Craig Topper via Phabricator via cfe-commits]

craig.topper added a comment.

What are we gaining from making the intrinsics use vector types if no vector operations are supported other than the intrinsics? Why can't we just use an xlen integer type?



================
Comment at: llvm/lib/Target/RISCV/RISCVISelLowering.cpp:706
+
+      setOperationAction(ISD::BITCAST, VT, Legal);
+
----------------
What about bitcast from float/double to any of these vector types? I'm guess that's not legal.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D99158