diff options
| author |   Lijian.Zhang <Lijian.Zhang@arm.com> | 2021-06-25 23:42:21 +0800 |
|---|---|---|
| committer |   Damjan Marion <dmarion@me.com> | 2021-07-13 21:24:04 +0000 |
| commit | cec484f080c06fde820de3a6695592deab21725f (patch) | |
| tree | 805e55bdc3eaa40c5940f3a7f3f67c4a79b59c22 | |
| parent | d154a17989b1da7abbfdb87b98b90cc5f4d3295f (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.h | 100 |
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 |