[PATCH 13/15][ARM/AArch64 Testsuite] Add float16 tests to advsimd-intrinsics testsuite

Kyrill Tkachov kyrylo.tkachov@arm.com
Tue Sep 8 10:56:00 GMT 2015


Hi all,

On 08/09/15 11:52, Alan Lawrence wrote:
> Ping. (Thanks, Christophe!).
>
> Original message: https://gcc.gnu.org/ml/gcc-patches/2015-07/msg02366.html
>
> On 25/08/15 14:28, Alan Lawrence wrote:
>> Christophe Lyon wrote:
>>> On 28 July 2015 at 13:26, Alan Lawrence <alan.lawrence@arm.com> wrote:
>>>> This is a respin of
>>>> https://gcc.gnu.org/ml/gcc-patches/2015-07/msg00488.html, fixing up the
>>>> testsuite for float16 vectors. Relative to the previous version, most of the
>>>> additions to the tests are now within #if..#endif such that they are only
>>>> compiled if we have a scalar __fp16 type (the exception is hfloat16_t: since
>>>> this is actually an integer type, we can define and use it without any
>>>> compiler fp16 support). Also we  try to use add_options_for_arm_neon_fp16
>>>> for all tests (on ARM targets), falling back to add_options_for_arm_neon if
>>>> the previous fails.
>>>>
>>>> Cross-tested on many multilibs, including -march=armv6,
>>>> -march=armv7-a{,-mfpu=neon-fp16}, -march=armv7-a/-mfpu=neon,
>>>> -march=armv7-a/-mfp16-format=none{,/-mfpu=neon-fp16,/-mfpu=neon},
>>>> -march=armv7-a/-mfp16-format=alternative .
>>>>
>>> Hi Alan,
>>>
>>> It looks OK.
>>> Did you also run the tests on AArch64?

arm-wise it looks ok to me since Christophe is ok with the changes.
Kyrill

>> Sorry, yes, I did - aarch64-none-linux-gnu, and aarch64_be-none-elf also.
>>
>> Thanks, Alan
>>



More information about the Gcc-patches mailing list