On Fri, 15 July 2022, 21:38 Peter Maydell, <peter.mayd...@linaro.org> wrote:

> > +DO_LD(q, _be, MO_128)
>
> +DO_LD(q, _le, MO_128)
>
> Coverity complains that these uses of MO_128 result in an
> array overrun for the pred_esz_masks[] array, because e.g.
> sme_ld1() calls sve_cont_ldst_elements() calls which uses esz
> as an index into pred_esz_masks[]. (Multiple coverity issues,
> affects both loads and stores.)
>
> Do we just need to add an extra entry to the array for
> MO_128 (presumably 0x0001000100010001ull) ?
>

Yes.  I thought I had done that, but perhaps merely meant to do so.  Thanks
in advance.


r~

Reply via email to