From 8c03fff40b05fa231adf8b564de22f03e42814b7 Mon Sep 17 00:00:00 2001 From: Johann Date: Tue, 8 Jul 2014 13:57:49 -0700 Subject: [PATCH] Move abs() to where it is effective vp8/encoder/x86/denoising_sse2.c:35:10: error: taking the absolute value of unsigned type 'unsigned int' has no effect [-Werror,-Wabsolute-value] Change-Id: I749ba8e6f55dbd9b822bfd4260a8397554f5e524 --- vp8/encoder/x86/denoising_sse2.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/vp8/encoder/x86/denoising_sse2.c b/vp8/encoder/x86/denoising_sse2.c index b84795ceb..3a4cf7ee7 100644 --- a/vp8/encoder/x86/denoising_sse2.c +++ b/vp8/encoder/x86/denoising_sse2.c @@ -30,9 +30,9 @@ static INLINE unsigned int abs_sum_diff_16x1(__m128i acc_diff) { _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 = _mm_cvtsi128_si32(hgfedcba); + unsigned int sum_diff = abs(_mm_cvtsi128_si32(hgfedcba)); - return abs(sum_diff); + return sum_diff; } int vp8_denoiser_filter_sse2(unsigned char *mc_running_avg_y, -- 2.40.0