[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v4 32/49] tests/tcg/arm: add fcvt test cases for
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [Qemu-devel] [PATCH v4 32/49] tests/tcg/arm: add fcvt test cases for AArch32/64 |
Date: |
Fri, 18 May 2018 11:53:47 -0300 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.7.0 |
On 05/18/2018 05:16 AM, Alex Bennée wrote:
>
> Philippe Mathieu-Daudé <address@hidden> writes:
>
>> On 05/17/2018 05:30 PM, Richard Henderson wrote:
>>> On 05/17/2018 10:47 AM, Alex Bennée wrote:
>>>> This runs through the usual float to float conversions and crucially
>>>> also runs with ARM Alternative Half Precision Format.
>>>>
>>>> Signed-off-by: Alex Bennée <address@hidden>
>>>> [rth: tweak vcvtb.f16.f32/vctb.f32.f16 code and regen]
>>>> Signed-off-by: Richard Henderson <address@hidden>
>>>>
>>>> ---
>>>> v4
>>>> - add fcvt.ref and check results against it
>>>> - fix single_to_half, single_to_double conversions
>>>> - properly toggle AHP mode (fpsr->fpcr)
>>>> - more values around the AHP margins
>>>> - add INF/NAN/SNAN inputs
>>>> - build for ARM and AArch64
>>>> - fix bug for hex literals
>>>> - add float-to-int
>>>> - checkpatch fix
>>>> ---
>>>
>>> Reviewed-by: Richard Henderson <address@hidden>
>>
>> Thanks Richard, I was not very motivated to review each line of the
>> fcvt.ref files =)
>>
>>> diff --git a/tests/tcg/arm/Makefile.target b/tests/tcg/arm/Makefile.target
>>> index 9d2b551732..7bb777f442 100644
>>> --- a/tests/tcg/arm/Makefile.target
>>> +++ b/tests/tcg/arm/Makefile.target
>>> @@ -8,7 +8,9 @@ ARM_SRC=$(SRC_PATH)/tests/tcg/arm
>>> # Set search path for all sources
>>> VPATH += $(ARM_SRC)
>>>
>>> -TESTS += hello-arm test-arm-iwmmxt
>>> +ARM_TESTS=hello-arm test-arm-iwmmxt
>>> +
>>> +TESTS += $(ARM_TESTS) fcvt
>>>
>>> hello-arm: CFLAGS+=-marm -ffreestanding
>>> hello-arm: LDFLAGS+=-nostdlib
>>> @@ -24,3 +26,14 @@ run-test-mmap: test-mmap
>>> $(call quiet-command, $(QEMU) -p 8192 $< 8192 > test-mmap-8192.out,
>> "TEST", "$< (8k pages) on $(TARGET_NAME)")
>>> $(call quiet-command, $(QEMU) -p 16384 $< 16384 >
>> test-mmap-16384.out, "TEST", "$< (16k pages) on $(TARGET_NAME)")
>>> endif
>>> +
>>> +ifeq ($(TARGET_NAME), arm)
>>> +fcvt: LDFLAGS+=-lm
>>> +# fcvt: CFLAGS+=-march=armv8.2-a+fp16 -mfpu=neon-fp-armv8
>>
>> Alex, What is your idea here, enable this later?
>> Maybe add some TODO comment around..
>
> How about:
>
> # AArch32 doesn't support all combinations of FP16<->FOO in it's base
> # profile. Once the AArch32 profile is caught up we should build an
> # additional binary and reference to test the additional combinations.
>
> I backed the change out when I realised the stretch compiler didn't
> support the same range as my local (18.04) gcc's.
Fine by me, thanks.
>>
>> Except this nit, for both Makefile.target:
>> Reviewed-by: Philippe Mathieu-Daudé <address@hidden>
- [Qemu-devel] [PATCH v4 49/49] tests/tcg: override runners for broken tests (!UPSTREAM), (continued)
- [Qemu-devel] [PATCH v4 49/49] tests/tcg: override runners for broken tests (!UPSTREAM), Alex Bennée, 2018/05/17
- [Qemu-devel] [PATCH v4 36/49] tests/tcg: enable building for s390x, Alex Bennée, 2018/05/17
- [Qemu-devel] [PATCH v4 47/49] Makefile.target: add (clean-/build-)guest-tests targets, Alex Bennée, 2018/05/17
- [Qemu-devel] [PATCH v4 25/49] tests/tcg/i386/test-i386: fix printf format, Alex Bennée, 2018/05/17
- [Qemu-devel] [PATCH v4 13/49] tests/tcg/multiarch: Build fix for linux-test, Alex Bennée, 2018/05/17
- [Qemu-devel] [PATCH v4 32/49] tests/tcg/arm: add fcvt test cases for AArch32/64, Alex Bennée, 2018/05/17