summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorLijian.Zhang <Lijian.Zhang@arm.com>2021-06-25 23:42:21 +0800
committerDamjan Marion <dmarion@me.com>2021-07-13 21:24:04 +0000
commitcec484f080c06fde820de3a6695592deab21725f (patch)
tree805e55bdc3eaa40c5940f3a7f3f67c4a79b59c22
parentd154a17989b1da7abbfdb87b98b90cc5f4d3295f (diff)
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
-rw-r--r--src/vppinfra/vector_neon.h100
1 files changed, 60 insertions, 40 deletions
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