From: Lijian.Zhang <Lijian.Zhang@arm.com>
Date: Fri, 25 Jun 2021 15:42:21 +0000 (+0800)
Subject: vppinfra: fix saturate add/sub NEON wrappers
X-Git-Tag: v22.02-rc0~207
X-Git-Url: https://gerrit.fd.io/r/gitweb?a=commitdiff_plain;h=cec484f080c06fde820de3a6695592deab21725f;p=vpp.git

vppinfra: fix saturate add/sub NEON wrappers

Fix the saturate add/sub wrappers in vector_neon.h by using the correct
intrinsics.

Type: fix

Signed-off-by: Lijian Zhang <Lijian.Zhang@arm.com>
Reviewed-by: Tianyu Li <Tianyu.Li@arm.com>
Change-Id: I38a85633948472d4bdb1c199a806633d3070013f
---

diff --git a/src/vppinfra/vector_neon.h b/src/vppinfra/vector_neon.h
index ffcbe702d24..70b05c60884 100644
--- a/src/vppinfra/vector_neon.h
+++ b/src/vppinfra/vector_neon.h
@@ -17,9 +17,6 @@
 #define included_vector_neon_h
 #include <arm_neon.h>
 
-/* Arithmetic */
-#define u16x8_sub_saturate(a,b) vsubq_u16(a,b)
-#define i16x8_sub_saturate(a,b) vsubq_s16(a,b)
 /* Dummy. Aid making uniform macros */
 #define vreinterpretq_u8_u8(a)  a
 /* Implement the missing intrinsics to make uniform macros */
@@ -54,43 +51,66 @@ u8x16_compare_byte_mask (u8x16 v)
 #define foreach_neon_vec128f \
   _(f,32,4,f32) _(f,64,2,f64)
 
-#define _(t, s, c, i) \
-static_always_inline t##s##x##c						\
-t##s##x##c##_splat (t##s x)						\
-{ return (t##s##x##c) vdupq_n_##i (x); }				\
-\
-static_always_inline t##s##x##c						\
-t##s##x##c##_load_unaligned (void *p)					\
-{ return (t##s##x##c) vld1q_##i (p); }					\
-\
-static_always_inline void						\
-t##s##x##c##_store_unaligned (t##s##x##c v, void *p)			\
-{ vst1q_##i (p, v); }							\
-\
-static_always_inline int						\
-t##s##x##c##_is_all_zero (t##s##x##c x)					\
-{ return !!(vminvq_u##s (vceqq_##i (vdupq_n_##i(0), x))); }						\
-\
-static_always_inline int						\
-t##s##x##c##_is_equal (t##s##x##c a, t##s##x##c b)			\
-{ return !!(vminvq_u##s (vceqq_##i (a, b))); }				\
-\
-static_always_inline int						\
-t##s##x##c##_is_all_equal (t##s##x##c v, t##s x)			\
-{ return t##s##x##c##_is_equal (v, t##s##x##c##_splat (x)); };		\
-\
-static_always_inline u32						\
-t##s##x##c##_zero_byte_mask (t##s##x##c x)			\
-{ uint8x16_t v = vreinterpretq_u8_u##s (vceqq_##i (vdupq_n_##i(0), x));  \
-  return u8x16_compare_byte_mask (v); } \
-\
-static_always_inline u##s##x##c						\
-t##s##x##c##_is_greater (t##s##x##c a, t##s##x##c b)			\
-{ return (u##s##x##c) vcgtq_##i (a, b); }				\
-\
-static_always_inline t##s##x##c						\
-t##s##x##c##_blend (t##s##x##c dst, t##s##x##c src, u##s##x##c mask)	\
-{ return (t##s##x##c) vbslq_##i (mask, src, dst); }
+#define _(t, s, c, i)                                                         \
+  static_always_inline t##s##x##c t##s##x##c##_splat (t##s x)                 \
+  {                                                                           \
+    return (t##s##x##c) vdupq_n_##i (x);                                      \
+  }                                                                           \
+                                                                              \
+  static_always_inline t##s##x##c t##s##x##c##_load_unaligned (void *p)       \
+  {                                                                           \
+    return (t##s##x##c) vld1q_##i (p);                                        \
+  }                                                                           \
+                                                                              \
+  static_always_inline void t##s##x##c##_store_unaligned (t##s##x##c v,       \
+							  void *p)            \
+  {                                                                           \
+    vst1q_##i (p, v);                                                         \
+  }                                                                           \
+                                                                              \
+  static_always_inline int t##s##x##c##_is_all_zero (t##s##x##c x)            \
+  {                                                                           \
+    return !!(vminvq_u##s (vceqq_##i (vdupq_n_##i (0), x)));                  \
+  }                                                                           \
+                                                                              \
+  static_always_inline int t##s##x##c##_is_equal (t##s##x##c a, t##s##x##c b) \
+  {                                                                           \
+    return !!(vminvq_u##s (vceqq_##i (a, b)));                                \
+  }                                                                           \
+  static_always_inline int t##s##x##c##_is_all_equal (t##s##x##c v, t##s x)   \
+  {                                                                           \
+    return t##s##x##c##_is_equal (v, t##s##x##c##_splat (x));                 \
+  };                                                                          \
+                                                                              \
+  static_always_inline u32 t##s##x##c##_zero_byte_mask (t##s##x##c x)         \
+  {                                                                           \
+    uint8x16_t v = vreinterpretq_u8_u##s (vceqq_##i (vdupq_n_##i (0), x));    \
+    return u8x16_compare_byte_mask (v);                                       \
+  }                                                                           \
+                                                                              \
+  static_always_inline u##s##x##c t##s##x##c##_is_greater (t##s##x##c a,      \
+							   t##s##x##c b)      \
+  {                                                                           \
+    return (u##s##x##c) vcgtq_##i (a, b);                                     \
+  }                                                                           \
+                                                                              \
+  static_always_inline t##s##x##c t##s##x##c##_add_saturate (t##s##x##c a,    \
+							     t##s##x##c b)    \
+  {                                                                           \
+    return (t##s##x##c) vqaddq_##i (a, b);                                    \
+  }                                                                           \
+                                                                              \
+  static_always_inline t##s##x##c t##s##x##c##_sub_saturate (t##s##x##c a,    \
+							     t##s##x##c b)    \
+  {                                                                           \
+    return (t##s##x##c) vqsubq_##i (a, b);                                    \
+  }                                                                           \
+                                                                              \
+  static_always_inline t##s##x##c t##s##x##c##_blend (                        \
+    t##s##x##c dst, t##s##x##c src, u##s##x##c mask)                          \
+  {                                                                           \
+    return (t##s##x##c) vbslq_##i (mask, src, dst);                           \
+  }
 
 foreach_neon_vec128i foreach_neon_vec128u