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.

Reply via email to