[PATCH] D114884: [VP] Strided loads/stores

[Simon Moll via Phabricator via llvm-commits]

simoll added inline comments.


================
Comment at: llvm/docs/LangRef.rst:19952
+
+    declare <4 x float> @llvm.experimental.vp.strided.load.v4f32.p0v4f32.i64(<4 x float>* %ptr, i64 %stride, <4 x i1> %mask, i32 %evl)
+    declare <vscale x 2 x i16> @llvm.experimental.vp.strided.load.nxv2i16.p0nxv2i16.i64(<vscale x 2 x i16>* %ptr, i64 %stride, <vscale x 2 x i1> %mask, i32 %evl)
----------------
loralb wrote:
> Reflecting on how the strided versions work, I was thinking if, semantically speaking, it would be better to define the pointer operand as a pointer to the scalar element type of the data/return operand instead of a pointer to a vector (this is only relevant while opaque pointers are not the default)
Agreed. Typed pointers will go away but we shouldn't be sloppy here.


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

https://reviews.llvm.org/D114884