Hi Andrea, > -----Original Message----- > From: Andrea Corallo <andrea.cora...@arm.com> > Sent: 02 November 2020 09:02 > To: gcc-patches@gcc.gnu.org > Cc: Kyrylo Tkachov <kyrylo.tkac...@arm.com>; Richard Earnshaw > <richard.earns...@arm.com>; nd <n...@arm.com> > Subject: [PATCH 4/x] arm: Add vst1_bf16 + vst1q_bf16 intrinsics > > Hi all, > > Forth patch of the serie here adding vst1_bf16, vst1q_bf16 bfloat16 > related neon intrinsics. > > Please see refer to: > ACLE <https://developer.arm.com/docs/101028/latest> > ISA <https://developer.arm.com/docs/ddi0596/latest> > > Regtested and bootstrapped. > > Thanks! > > Andrea > > gcc/ChangeLog > > 2020-10-29 Andrea Corallo <andrea.cora...@arm.com> > > * config/arm/arm_neon.h (vst1_bf16, vst1q_bf16): Add intrinsics. > * config/arm/arm_neon_builtins.def : Touch for: > __builtin_neon_vst1v4bf, __builtin_neon_vst1v8bf. >
I see this patch also has the hunk: diff --git a/gcc/config/arm/arm-builtins.c b/gcc/config/arm/arm-builtins.c index 33e8015b140..6dc5df93216 100644 --- a/gcc/config/arm/arm-builtins.c +++ b/gcc/config/arm/arm-builtins.c @@ -946,6 +946,9 @@ typedef struct { #define VAR13(T, N, A, B, C, D, E, F, G, H, I, J, K, L, M) \ VAR12 (T, N, A, B, C, D, E, F, G, H, I, J, K, L) \ VAR1 (T, N, M) +#define VAR14(T, N, A, B, C, D, E, F, G, H, I, J, K, L, M, O) \ + VAR13 (T, N, A, B, C, D, E, F, G, H, I, J, K, L, M) \ + VAR1 (T, N, O) /* The builtin data can be found in arm_neon_builtins.def, arm_vfp_builtins.def and arm_acle_builtins.def. The entries in arm_neon_builtins.def require That is a duplicate from a previous patch in the series and doesn't appear in the ChangeLog here. I think it's in here by mistake? Otherwise ok. Thanks, Kyrill > gcc/testsuite/ChangeLog > > 2020-10-29 Andrea Corallo <andrea.cora...@arm.com> > > * gcc.target/arm/simd/vst1_bf16_1.c: New test.