Re: [PATCH v5 26/60] target/riscv: vector widening saturating scaled multiply-add

[Richard Henderson]

On 3/12/20 7:58 AM, LIU Zhiwei wrote:
> +static uint16_t vwsmaccu8(CPURISCVState *env, uint8_t a, uint8_t b,
> +    uint16_t c)
> +{
> +    uint8_t round;
> +    uint16_t res = (uint16_t)a * (uint16_t)b;
> +
> +    round = get_round(env, res, 4);
> +    res   = (res >> 4) + round;
> +    return saddu16(env, c, res);
> +}
> +static uint32_t vwsmaccu16(CPURISCVState *env, uint16_t a, uint16_t b,

With the same caveat for vxrm as before, and the spacing.

r~