[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 13/42] target/arm: Convert "single-precision" re
From: |
Richard Henderson |
Subject: |
Re: [Qemu-devel] [PATCH 13/42] target/arm: Convert "single-precision" register moves to decodetree |
Date: |
Fri, 7 Jun 2019 13:08:21 -0500 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.7.0 |
On 6/6/19 12:45 PM, Peter Maydell wrote:
> Convert the "single-precision" register moves to decodetree:
> * VMSR
> * VMRS
> * VMOV between general purpose register and single precision
>
> Note that the VMSR/VMRS conversions make our handling of
> the "should this UNDEF?" checks consistent between the two
> instructions:
> * VMSR to MVFR0, MVFR1, MVFR2 now UNDEF from EL0
> (previously was a nop)
> * VMSR to FPSID now UNDEFs from EL0 or if VFPv3 or better
> (previously was a nop)
> * VMSR to FPINST and FPINST2 now UNDEF if VFPv3 or better
> (previously would write to the register, which had no
> guest-visible effect because we always UNDEF reads)
>
> We also tighten up the decode: we were previously underdecoding
> some SBZ or SBO bits.
>
> The conversion of VMOV_single includes the expansion out of the
> gen_mov_F0_vreg()/gen_vfp_mrs() and gen_mov_vreg_F0()/gen_vfp_msr()
> sequences into the simpler direct load/store of the TCG temp via
> neon_{load,store}_reg32(): we know in the new function that we're
> always single-precision, we don't need to use the old-and-deprecated
> cpu_F0* TCG globals, and we don't happen to have the declaration of
> gen_vfp_msr() and gen_vfp_mrs() at the point in the file where the
> new function is.
>
> Signed-off-by: Peter Maydell <address@hidden>
> ---
> target/arm/translate-vfp.inc.c | 161 +++++++++++++++++++++++++++++++++
> target/arm/translate.c | 148 +-----------------------------
> target/arm/vfp.decode | 4 +
> 3 files changed, 168 insertions(+), 145 deletions(-)
Reviewed-by: Richard Henderson <address@hidden>
r~
- Re: [Qemu-devel] [PATCH 07/42] target/arm: Convert VMINNM, VMAXNM to decodetree, (continued)
- [Qemu-devel] [PATCH 39/42] target/arm: Convert VJCVT to decodetree, Peter Maydell, 2019/06/06
- [Qemu-devel] [PATCH 32/42] target/arm: Convert VMOV (register) to decodetree, Peter Maydell, 2019/06/06
- [Qemu-devel] [PATCH 35/42] target/arm: Convert the VCVT-to-f16 insns to decodetree, Peter Maydell, 2019/06/06
- [Qemu-devel] [PATCH 27/42] target/arm: Convert VFP fused multiply-add insns to decodetree, Peter Maydell, 2019/06/06
- [Qemu-devel] [PATCH 13/42] target/arm: Convert "single-precision" register moves to decodetree, Peter Maydell, 2019/06/06
- Re: [Qemu-devel] [PATCH 13/42] target/arm: Convert "single-precision" register moves to decodetree,
Richard Henderson <=
- [Qemu-devel] [PATCH 18/42] target/arm: Convert VFP VMLA to decodetree, Peter Maydell, 2019/06/06
- [Qemu-devel] [PATCH 28/42] target/arm: Convert VMOV (imm) to decodetree, Peter Maydell, 2019/06/06
[Qemu-devel] [PATCH 41/42] target/arm: Convert float-to-integer VCVT insns to decodetree, Peter Maydell, 2019/06/06
[Qemu-devel] [PATCH 42/42] target/arm: Fix short-vector increment behaviour, Peter Maydell, 2019/06/06