On 8/24/20 7:29 AM, Peter Maydell wrote: > The aa32_fp16_arith feature check function currently looks at the > AArch64 ID_AA64PFR0 register. This is (as the comment notes) not > correct. The bogus check was put in mostly to allow testing of the > fp16 variants of the VCMLA instructions and it was something of > a mistake that we allowed them to exist in master. > > Switch the feature check function to testing VMFR1.FPHP, which is > what it ought to be. > > This will remove emulation of the VCMLA and VCADD insns from > AArch32 code running on an AArch64 '-cpu max' using system emulation. > (They were never enabled for aarch32 linux-user and system-emulation.) > Since we weren't advertising their existence via the AArch32 ID > register, well-behaved guests wouldn't have been using them anyway. > > Once we have implemented all the AArch32 support for the FP16 extension > we will advertise it in the MVFR1 ID register field, which will reenable > these insns along with all the others. > > Signed-off-by: Peter Maydell <peter.mayd...@linaro.org> > --- > I don't expect that any guests will have been using these insns, > but in any case the fp16 work will be all done before the next > QEMU release and the insns re-enabled... > --- > target/arm/cpu.h | 7 +------ > 1 file changed, 1 insertion(+), 6 deletions(-)
Cc qemu-stable, for the bug fix? Reviewed-by: Richard Henderson <richard.hender...@linaro.org> r~