[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 49/55] target/arm: Implement MVE VQDMULL (vector), (continued)
- [PATCH 48/55] target/arm: Implement MVE VQDMLSDH and VQRDMLSDH, Peter Maydell, 2021/06/07
- [PATCH 50/55] target/arm: Implement MVE VRHADD, Peter Maydell, 2021/06/07
- [PATCH 52/55] target/arm: Implement MVE VCADD, Peter Maydell, 2021/06/07
- [PATCH 53/55] target/arm: Implement MVE VHCADD, Peter Maydell, 2021/06/07
- [PATCH 55/55] target/arm: Make VMOV scalar <-> gpreg beatwise for MVE, Peter Maydell, 2021/06/07
- Re: [PATCH 00/55] target/arm: First slice of MVE implementation, no-reply, 2021/06/09