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

[Kyrill Tkachov <kyrylo dot tkachov at arm dot com>]

On 08/09/15 11:55, Kyrill Tkachov wrote:
> 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.

To be clear, this is ok for trunk.

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