Skip to content

Commit e7dc5d6

Browse files
committed
Some tweaks to the SVE2p1 load and store intrinsics
The pre-SVE2p1 gather and scatter intrinsics allow vector displacements (offsets or indices) to be either signed or unsigned. svld1q and svst1q instead required them to be unsigned. This patch adds signed versions too, for consistency. Also, the SVE2p1 stores were specified to take pointers to const, but they ought to be pointers to non-const instead.
1 parent 36736ea commit e7dc5d6

File tree

1 file changed

+11
-6
lines changed

1 file changed

+11
-6
lines changed

main/acle.md

Lines changed: 11 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -423,6 +423,7 @@ Armv8.4-A [[ARMARMv84]](#ARMARMv84). Support is added for the Dot Product intrin
423423
* Fixed range of operand `o0` (too small) in AArch64 system register designations.
424424
* Fixed SVE2.1 quadword gather load/scatter store intrinsics.
425425
* Removed unnecessary Zd argument from `svcvtnb_mf8[_f32_x2]_fpm`.
426+
* Removed extraneous `const` from SVE2.1 store intrinsics.
426427

427428
### References
428429

@@ -9182,11 +9183,13 @@ Gather Load Quadword.
91829183
// _mf8, _bf16, _f16, _f32, _f64
91839184
svint8_t svld1q_gather[_u64base]_s8(svbool_t pg, svuint64_t zn);
91849185
svint8_t svld1q_gather[_u64base]_offset_s8(svbool_t pg, svuint64_t zn, int64_t offset);
9186+
svint8_t svld1q_gather_[s64]offset[_s8](svbool_t pg, const int8_t *base, svint64_t offset);
91859187
svint8_t svld1q_gather_[u64]offset[_s8](svbool_t pg, const int8_t *base, svuint64_t offset);
91869188

91879189
// Variants are also available for:
91889190
// _u16, _u32, _s32, _u64, _s64
91899191
// _bf16, _f16, _f32, _f64
9192+
svint16_t svld1q_gather_[s64]index[_s16](svbool_t pg, const int16_t *base, svint64_t index);
91909193
svint16_t svld1q_gather_[u64]index[_s16](svbool_t pg, const int16_t *base, svuint64_t index);
91919194
svint16_t svld1q_gather[_u64base]_index_s16(svbool_t pg, svuint64_t zn, int64_t index);
91929195
```
@@ -9256,14 +9259,14 @@ Contiguous store of single vector operand, truncating from quadword.
92569259
``` c
92579260
// Variants are also available for:
92589261
// _u32, _s32
9259-
void svst1wq[_f32](svbool_t, const float32_t *ptr, svfloat32_t data);
9260-
void svst1wq_vnum[_f32](svbool_t, const float32_t *ptr, int64_t vnum, svfloat32_t data);
9262+
void svst1wq[_f32](svbool_t, float32_t *ptr, svfloat32_t data);
9263+
void svst1wq_vnum[_f32](svbool_t, float32_t *ptr, int64_t vnum, svfloat32_t data);
92619264

92629265

92639266
// Variants are also available for:
92649267
// _u64, _s64
9265-
void svst1dq[_f64](svbool_t, const float64_t *ptr, svfloat64_t data);
9266-
void svst1dq_vnum[_f64](svbool_t, const float64_t *ptr, int64_t vnum, svfloat64_t data);
9268+
void svst1dq[_f64](svbool_t, float64_t *ptr, svfloat64_t data);
9269+
void svst1dq_vnum[_f64](svbool_t, float64_t *ptr, int64_t vnum, svfloat64_t data);
92679270
```
92689271

92699272
#### ST1Q
@@ -9276,12 +9279,14 @@ Scatter store quadwords.
92769279
// _mf8, _bf16, _f16, _f32, _f64
92779280
void svst1q_scatter[_u64base][_s8](svbool_t pg, svuint64_t zn, svint8_t data);
92789281
void svst1q_scatter[_u64base]_offset[_s8](svbool_t pg, svuint64_t zn, int64_t offset, svint8_t data);
9279-
void svst1q_scatter_[u64]offset[_s8](svbool_t pg, const uint8_t *base, svuint64_t offset, svint8_t data);
9282+
void svst1q_scatter_[s64]offset[_s8](svbool_t pg, uint8_t *base, svint64_t offset, svint8_t data);
9283+
void svst1q_scatter_[u64]offset[_s8](svbool_t pg, uint8_t *base, svuint64_t offset, svint8_t data);
92809284

92819285
// Variants are also available for:
92829286
// _u16, _u32, _s32, _u64, _s64
92839287
// _bf16, _f16, _f32, _f64
9284-
void svst1q_scatter_[u64]index[_s16](svbool_t pg, const int16_t *base, svuint64_t index, svint16_t data);
9288+
void svst1q_scatter_[s64]index[_s16](svbool_t pg, int16_t *base, svint64_t index, svint16_t data);
9289+
void svst1q_scatter_[u64]index[_s16](svbool_t pg, int16_t *base, svuint64_t index, svint16_t data);
92859290
void svst1q_scatter[_u64base]_index[_s16](svbool_t pg, svuint64_t zn, int64_t index, svint16_t data);
92869291
```
92879292

0 commit comments

Comments
 (0)