From 837aefa3a281f02fd55f2b3f2cf4de8c579e183d Mon Sep 17 00:00:00 2001 From: Rakesh Kumar Date: Thu, 9 Aug 2018 22:35:46 +0530 Subject: [PATCH] Fix Compiler warnings Intrinsic _mm_set_epi8 require char input, converting everything to char. Including log.h from android library instead of cutils. Bug: 112316276 Test: before/after object comparison for no diff in generated code. Change-Id: I82b8c48494619b64465eda2476c7c903652c6066 --- decoder/ih264d_dpb_mgr.c | 2 +- encoder/x86/ih264e_intra_modes_eval_ssse3.c | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/decoder/ih264d_dpb_mgr.c b/decoder/ih264d_dpb_mgr.c index b7ee498..e35cb25 100644 --- a/decoder/ih264d_dpb_mgr.c +++ b/decoder/ih264d_dpb_mgr.c @@ -19,7 +19,7 @@ */ #ifdef __ANDROID__ #include "log/log.h" -#include +#include #endif #include "ih264_typedefs.h" #include "ih264_macros.h" diff --git a/encoder/x86/ih264e_intra_modes_eval_ssse3.c b/encoder/x86/ih264e_intra_modes_eval_ssse3.c index c11d7f2..ea8a6c8 100644 --- a/encoder/x86/ih264e_intra_modes_eval_ssse3.c +++ b/encoder/x86/ih264e_intra_modes_eval_ssse3.c @@ -574,7 +574,7 @@ void ih264e_evaluate_intra_4x4_modes_ssse3(UWORD8 *pu1_src, pred2_16x8b = _mm_set1_epi8(dcval); } else - pred2_16x8b = _mm_set1_epi8(128); + pred2_16x8b = _mm_set1_epi8(-128); sad_8x16b = _mm_sad_epu8(src_16x8b, pred2_16x8b); @@ -1135,7 +1135,7 @@ void ih264e_evaluate_intra_chroma_modes_ssse3(UWORD8 *pu1_src, } else { - pred1_16x8b = _mm_set1_epi8(128); + pred1_16x8b = _mm_set1_epi8(-128); tmp1_8x16b = _mm_sad_epu8(src1_16x8b, pred1_16x8b); tmp2_8x16b = _mm_sad_epu8(src2_16x8b, pred1_16x8b);