On 17 February 2018 at 18:23, Richard Henderson <richard.hender...@linaro.org> wrote: > Signed-off-by: Richard Henderson <richard.hender...@linaro.org> > --- > target/arm/helper-sve.h | 4 +++ > target/arm/sve_helper.c | 70 > ++++++++++++++++++++++++++++++++++++++++++++++ > target/arm/translate-sve.c | 26 +++++++++++++++++ > target/arm/sve.decode | 3 ++ > 4 files changed, 103 insertions(+)
> +/* FP Trig Multiply-Add. */ > + > +void HELPER(sve_ftmad_h)(void *vd, void *vn, void *vm, void *vs, uint32_t > desc) > +{ > + static const float16 coeff[16] = { > + 0x3c00, 0xb155, 0x2030, 0x0000, 0x0000, 0x0000, 0x0000, 0x0000, > + 0x3c00, 0xb800, 0x293a, 0x0000, 0x0000, 0x0000, 0x0000, 0x0000, > + }; Comment that these are constants from the pseudocode, or whatever we agreed for the earlier patch with constant tables... Reviewed-by: Peter Maydell <peter.mayd...@linaro.org> thanks -- PMM