Re: [PATCH v2 3/4] target/riscv: Encode the FS and VS on a normal way for tb flags

[Richard Henderson]

On 3/24/23 07:30, LIU Zhiwei wrote:
> Reuse the MSTATUS_FS and MSTATUS_VS for the tb flags positions is not a
> normal way.
>
> It will make it hard to change the tb flags layout. And even worse, if we
> want to keep tb flags for a same extension togather without a hole.
>
> Signed-off-by: LIU Zhiwei <zhiwei_liu@linux.alibaba.com>
> Reviewed-by: Weiwei Li <liweiwei@iscas.ac.cn>

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

> +        flags =  FIELD_DP32(flags, TB_FLAGS, FS,
> +        flags =  FIELD_DP32(flags, TB_FLAGS, VS,

Extra space after =.


r~