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=2c55e6caa9432b2c1f081cb3aeddd36abec03233
https://gcc.gnu.org/git/?p=gcc.git;a=commit;h=a4004f62d60ada3a20dbf30146ca461047a575cc

and to the gcc-9 branch:
https://gcc.gnu.org/git/?p=gcc.git;a=commit;h=c5aca0333b723d5e20366593cd01047d105f54e4

Sebastian

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

Reply via email to