Re: [PATCH v3] target/arm: Fix sve2 ldnt1 and stnt1

2022-03-10 Thread Peter Maydell
On Tue, 8 Mar 2022 at 11:56, Peter Maydell wrote: > > Anyway > Reviewed-by: Peter Maydell ...and applied to target-arm.next. -- PMM

Re: [PATCH v3] target/arm: Fix sve2 ldnt1 and stnt1

2022-03-08 Thread Richard Henderson
On 3/8/22 01:56, Peter Maydell wrote: -# SVE2 64-bit gather non-temporal load -# (scalar plus unpacked 32-bit unscaled offsets) +# SVE2 64-bit gather non-temporal load (scalar plus 64-bit unscaled offsets) LDNT1_zprz 1100010 msz:2 00 rm:5 1 u:1 0 pg:3 rn:5 rd:5 \ -&rprr_g

Re: [PATCH v3] target/arm: Fix sve2 ldnt1 and stnt1

2022-03-08 Thread Peter Maydell
On Tue, 8 Mar 2022 at 03:18, Richard Henderson wrote: > > For both ldnt1 and stnt1, the meaning of the Rn and Rm are different > from ld1 and st1: the vector and integer registers are reversed, and > the integer register 31 refers to XZR instead of SP. > > Secondly, the 64-bit version of ldnt1 was

[PATCH v3] target/arm: Fix sve2 ldnt1 and stnt1

2022-03-07 Thread Richard Henderson
For both ldnt1 and stnt1, the meaning of the Rn and Rm are different from ld1 and st1: the vector and integer registers are reversed, and the integer register 31 refers to XZR instead of SP. Secondly, the 64-bit version of ldnt1 was being interpreted as 32-bit unpacked unscaled offset instead of 6