On Tue, Jan 14, 2020 at 11:29 PM Richard Henderson <
richard.hender...@linaro.org> wrote:
> On 1/14/20 12:13 AM, shihpo.h...@sifive.com wrote:
> > if (riscv_cpu_fp_enabled(env)) {
> > - *flags |= TB_FLAGS_MSTATUS_FS;
> > + *flags |= env->mstatus & MSTATUS_FS;
> > }
>
> Note that riscv_cpu_fp_enabled is *also* testing env->mstatus, and that
> this
> assignment can be made unconditional.
>
> Otherwise,
> Reviewed-by: Richard Henderson <richard.hender...@linaro.org>
>
I'll send v3 that cleans up the patch accordingly.
Thanks for the review!
