Hi Alfie, > On 3 Jul 2024, at 12:10, alfie.richa...@arm.com wrote: > > External email: Use caution opening links or attachments > > > From: Alfie Richards <alfie.richa...@arm.com> > > Hi All, > > This fixes a case where the operands for the addp NEON intrinsic were > erroneously swapped. > > Regtested on aarch64-unknown-linux-gnu > > Ok for master and GCC14.2?
This is okay, though I would have done it as a single commit rather than splitting the test case separately and then updating it to remove the xfail. But it’s okay as is anyway. Do you need someone to commit it for you? Thanks, Kyrill > > Kind regards, > Alfie Richards > > Alfie Richards (2): > Aarch64: Add test for non-commutative SIMD intrinsic > Aarch64, bugfix: Fix NEON bigendian addp intrinsic [PR114890] > > gcc/config/aarch64/aarch64-simd.md | 2 - > .../aarch64/vector_intrinsics_asm.c | 371 ++++++++++++++++++ > 2 files changed, 371 insertions(+), 2 deletions(-) > create mode 100644 gcc/testsuite/gcc.target/aarch64/vector_intrinsics_asm.c > > -- > 2.34.1 >
