diff options
| author | James Zern <jzern@google.com> | 2024-03-07 17:39:12 -0800 |
|---|---|---|
| committer | James Zern <jzern@google.com> | 2024-03-13 18:36:14 +0000 |
| commit | c1494fa57e5d73d25334a6c2293559c8907b560c (patch) | |
| tree | fc6fa7287aff68d5a05b29e9844eb1ba61536797 | |
| parent | daa33cca37e346861fa3b7915857eb8871e0caf0 (diff) | |
| download | libvpx-c1494fa57e5d73d25334a6c2293559c8907b560c.tar.gz | |
neon: fix -Woverflow warnings
with signed char values, 128 -> -128
Change-Id: Iec2257729d7878459794d6a3d6bc3f745d39e97c
| -rw-r--r-- | vp8/common/arm/neon/sixtappredict_neon.c | 2 | ||||
| -rw-r--r-- | vpx_dsp/arm/loopfilter_neon.c | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/vp8/common/arm/neon/sixtappredict_neon.c b/vp8/common/arm/neon/sixtappredict_neon.c index ee3c281f0..a54e81084 100644 --- a/vp8/common/arm/neon/sixtappredict_neon.c +++ b/vp8/common/arm/neon/sixtappredict_neon.c @@ -16,7 +16,7 @@ #include "vpx_ports/mem.h" static const int8_t vp8_sub_pel_filters[8][8] = { - { 0, 0, 128, 0, 0, 0, 0, 0 }, /* note that 1/8 pel positions are */ + { 0, 0, -128, 0, 0, 0, 0, 0 }, /* note that 1/8 pel positions are */ { 0, -6, 123, 12, -1, 0, 0, 0 }, /* just as per alpha -0.5 bicubic */ { 2, -11, 108, 36, -8, 1, 0, 0 }, /* New 1/4 pel 6 tap filter */ { 0, -9, 93, 50, -6, 0, 0, 0 }, diff --git a/vpx_dsp/arm/loopfilter_neon.c b/vpx_dsp/arm/loopfilter_neon.c index c54e58823..579096d78 100644 --- a/vpx_dsp/arm/loopfilter_neon.c +++ b/vpx_dsp/arm/loopfilter_neon.c @@ -162,7 +162,7 @@ FUN_FLIP_SIGN(16, q_) // flip_sign_16 #define FUN_FLIP_SIGN_BACK(w, r) \ static INLINE uint8x##w##_t flip_sign_back_##w(const int8x##w##_t v) { \ - const int8x##w##_t sign_bit = vdup##r##n_s8(0x80); \ + const int8x##w##_t sign_bit = vdup##r##n_s8((int8_t)0x80); \ return vreinterpret##r##u8_s8(veor##r##s8(v, sign_bit)); \ } |