[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 52/55] target/arm: Implement MVE VCADD
From: |
Peter Maydell |
Subject: |
Re: [PATCH 52/55] target/arm: Implement MVE VCADD |
Date: |
Thu, 10 Jun 2021 20:16:49 +0100 |
On Wed, 9 Jun 2021 at 22:16, Richard Henderson
<richard.henderson@linaro.org> wrote:
>
> On 6/7/21 9:58 AM, Peter Maydell wrote:
> > +/*
> > + * VCADD Qd == Qm at size MO_32 is UNPREDICTABLE; we choose not to diagnose
> > + * so we can reuse the DO_2OP macro. (Our implementation calculates the
> > + * "expected" results in this case.)
> > + */
> You've done this elsewhere, though.
Yeah, because in those cases the op had to have its own hand-written
trans_ function for other reasons so the check was easy to add. Hence
the comment about why this particular case doesn't do that.
> Either way,
> Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
thanks
-- PMM
- [PATCH 50/55] target/arm: Implement MVE VRHADD, (continued)
- [PATCH 51/55] target/arm: Implement MVE VADC, VSBC, Peter Maydell, 2021/06/07
- [PATCH 52/55] target/arm: Implement MVE VCADD, Peter Maydell, 2021/06/07
- [PATCH 55/55] target/arm: Make VMOV scalar <-> gpreg beatwise for MVE, Peter Maydell, 2021/06/07
- [PATCH 53/55] target/arm: Implement MVE VHCADD, Peter Maydell, 2021/06/07
- [PATCH 22/55] target/arm: Implement MVE VADD, VSUB, VMUL, Peter Maydell, 2021/06/07
- [PATCH 43/55] target/arm: Implement MVE VQSHL (vector), Peter Maydell, 2021/06/07
- [PATCH 24/55] target/arm: Implement MVE VRMULH, Peter Maydell, 2021/06/07