Re: [COMMITTED][PATCH PR63173] [AARCH64, NEON] Improve vld[234](q?)_dup intrinsics

[Alan Lawrence <alan dot lawrence at arm dot com>]

So we've been seeing

FAIL: gcc.target/aarch64/vldN_dup_1.c

on aarch64_be-none-elf, since this patch went in. Felix, did you test for bigendian?

However, this failure is fixed if I apply David Sherwood's patch set:

https://gcc.gnu.org/ml/gcc-patches/2014-10/msg00942.html
https://gcc.gnu.org/ml/gcc-patches/2014-10/msg01099.html

So maybe I can "ping" that on David's behalf (+Alan Hayward's 
https://gcc.gnu.org/ml/gcc-patches/2014-10/msg00952.html on which those two 
patches depend).

Cheers, Alan



Yangfei (Felix) wrote:
> > On 24 October 2014 03:21, Yangfei (Felix) <felix.yang@huawei.com> wrote:
> >
> > > Thanks for the comments. I updated the patch with the intrinsic moved to its
> > place.
> > > Attached please find the new version of the patch.
> > > OK for the trunk?
> > >
> > >
> > > Index: gcc/ChangeLog
> > =============================================================
> > ======
> > > --- gcc/ChangeLog       (revision 216558)
> > > +++ gcc/ChangeLog       (working copy)
> > > @@ -1,3 +1,39 @@
> > > +2014-10-23  Felix Yang  <felix.yang@huawei.com>
> > > +       Jiji Jiang <jiangjiji@huawei.com>
> > Double space before "<".  Otherwise OK.
> > Thanks /Marcus
>
> Committed to trunk as r216630 with the space issue fixed. Thanks.