Re: [PATCH v3 13/20] target/riscv: Fix RESERVED field length in VTYPE

[Richard Henderson]

On 11/11/21 12:33 PM, Richard Henderson wrote:
> On 11/11/21 6:57 AM, LIU Zhiwei wrote:
> > Signed-off-by: LIU Zhiwei <zhiwei_liu@c-sky.com>
> > ---
> >   target/riscv/cpu.h | 2 +-
> >   1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/target/riscv/cpu.h b/target/riscv/cpu.h
> > index 52ce670cbe..b48c7c346c 100644
> > --- a/target/riscv/cpu.h
> > +++ b/target/riscv/cpu.h
> > @@ -105,7 +105,7 @@ typedef struct CPURISCVState CPURISCVState;
> >   FIELD(VTYPE, VLMUL, 0, 2)
> >   FIELD(VTYPE, VSEW, 2, 3)
> >   FIELD(VTYPE, VEDIV, 5, 2)
> > -FIELD(VTYPE, RESERVED, 7, sizeof(target_ulong) * 8 - 9)
> > +FIELD(VTYPE, RESERVED, 7, sizeof(target_ulong) * 8 - 8)
>
> Or better, remove it in the next patch.

Bah, nevermind.
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>


r~