diff options
author | James Zern <jzern@google.com> | 2022-07-27 15:22:37 -0700 |
---|---|---|
committer | James Zern <jzern@google.com> | 2022-07-27 16:59:21 -0700 |
commit | 4667992d8ba51d60045d6fed705635a6455eb4f8 (patch) | |
tree | e7b269b6f46dcb936cfe1c3d832bb96a08255e0d /vp8 | |
parent | e533d989ea93ee6cd980cc07e077475733694687 (diff) | |
download | libvpx-4667992d8ba51d60045d6fed705635a6455eb4f8.tar libvpx-4667992d8ba51d60045d6fed705635a6455eb4f8.tar.gz libvpx-4667992d8ba51d60045d6fed705635a6455eb4f8.tar.bz2 libvpx-4667992d8ba51d60045d6fed705635a6455eb4f8.zip |
x86: normalize type with _mm_cvtsi128_si32
prefer int in most cases
w/clang -fsanitize=integer fixes warnings of the form:
implicit conversion from type 'int' of value -809931979 (32-bit, signed)
to type 'uint32_t' (aka 'unsigned int') changed the value to 3485035317
(32-bit, unsigned)
Bug: b/229626362
Change-Id: I0c6604efc188f2660c531eddfc7aa10060637813
Diffstat (limited to 'vp8')
-rw-r--r-- | vp8/encoder/x86/denoising_sse2.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/vp8/encoder/x86/denoising_sse2.c b/vp8/encoder/x86/denoising_sse2.c index 89cad5335..f35b93016 100644 --- a/vp8/encoder/x86/denoising_sse2.c +++ b/vp8/encoder/x86/denoising_sse2.c @@ -30,7 +30,7 @@ static INLINE unsigned int abs_sum_diff_16x1(__m128i acc_diff) { _mm_add_epi32(hg_fe_dc_ba, _mm_srli_si128(hg_fe_dc_ba, 8)); const __m128i hgfedcba = _mm_add_epi32(hgfe_dcba, _mm_srli_si128(hgfe_dcba, 4)); - unsigned int sum_diff = abs(_mm_cvtsi128_si32(hgfedcba)); + unsigned int sum_diff = (unsigned int)abs(_mm_cvtsi128_si32(hgfedcba)); return sum_diff; } |