shithub: libvpx

Info  •  Files  •  Log  •  Branches

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

--- a/vp8/encoder/x86/denoising_sse2.c

+++ b/vp8/encoder/x86/denoising_sse2.c

@@ -30,9 +30,9 @@

                                           _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,