> -----Original Message----- > From: Christophe Lyon <christophe.l...@linaro.org> > Sent: 16 September 2020 13:43 > To: Pop, Sebastian <s...@amazon.com> > Cc: Kyrylo Tkachov <kyrylo.tkac...@arm.com>; gcc-patches@gcc.gnu.org > Subject: Re: [aarch64] Backport missing NEON intrinsics to GCC8 > > On Wed, 16 Sep 2020 at 07:21, Pop, Sebastian via Gcc-patches > <gcc-patches@gcc.gnu.org> wrote: > > > > Thanks Kyrill for your review. > > > > I committed the patches to the gcc-8 branch: > > > https://gcc.gnu.org/git/?p=gcc.git;a=commitdiff;h=2c55e6caa9432b2c1f081 > cb3aeddd36abec03233 > > > https://gcc.gnu.org/git/?p=gcc.git;a=commit;h=a4004f62d60ada3a20dbf301 > 46ca461047a575cc > > > > and to the gcc-9 branch: > > > https://gcc.gnu.org/git/?p=gcc.git;a=commit;h=c5aca0333b723d5e2036659 > 3cd01047d105f54e4 > > > > Sebastian > > > > Hi Sebastian, > > The new tests vld1x3 and vld1x4 fail on arm, I am seeing new failures > on gcc-8 and gcc-9 branches > after r8-10451, r8-10452 and r9-8874. > Is that expected/fixed later in the backport series? > (on the release branches, my validations are running for every commit)
Hmm, IIRC they're not implemented for arm IIRC so they should be xfailed or skipped on arm. Kyrill > > Thanks, > > Christophe > > > > On 9/15/20, 7:46 AM, "Kyrylo Tkachov" <kyrylo.tkac...@arm.com> wrote: > > > > Hi Sebastian, > > > > This patch implements missing intrinsics. > > I'm okay with this being applied to the GCC 8 branch as these intrinsics > have been defined in ACLE for a long time. > > It is arguably a bug that they've been missing from GCC8. > > Their implementation is fairly self-contained we haven't had any bugs > reported against these in my recollection. > > > > So ok on the grounds that it's a bug-fix. > > Thanks, > > Kyrill > > > > From: Pop, Sebastian <s...@amazon.com> > > Sent: 11 September 2020 20:54 > > To: gcc-patches@gcc.gnu.org; Kyrylo Tkachov > <kyrylo.tkac...@arm.com> > > Subject: [aarch64] Backport missing NEON intrinsics to GCC8 > > > > Hi, > > > > gcc-8 branch is missing NEON intrinsics for loads and stores. > > Attached patches pass bootstrap and regression testing on Graviton2 > aarch64-linux. > > > > Ok to commit to gcc-8 branch? > > > > Thanks, > > Sebastian > >
