commit ea1290213475252d4d259edf58d5ba1c6dc7b372
author James Zern <jzern@google.com> Tue Aug 30 17:17:01 2022 -0700
committer James Zern <jzern@google.com> Wed Aug 31 18:22:09 2022 +0000
tree 5aacd6a05ff5afc7996380d90972f2d2a8b2f5e3
parent f0138246e839bdf87c7a6430baf99fc5520a1b8c

wedge_utils_{avx2,sse2}: fix implicit conversion warnings

w/clang -fsanitize=integer quiets warnings of the form:
implicit conversion from type 'uint64_t' (aka 'unsigned long') of value
18446744073666365769 (64-bit, unsigned) to type 'int64_t' (aka 'long')
changed the value to -43185847 (64-bit, signed)

Bug: aomedia:3136
Bug: b/229626362
Change-Id: Iaae6a48a1e526868a9218598a6e34fbc55a25d1a

av1/encoder/x86/wedge_utils_avx2.c

diff --git a/av1/encoder/x86/wedge_utils_avx2.c b/av1/encoder/x86/wedge_utils_avx2.c
index 665951c..bbc62d5 100644
--- a/av1/encoder/x86/wedge_utils_avx2.c
+++ b/av1/encoder/x86/wedge_utils_avx2.c
@@ -142,7 +142,7 @@
   v_acc_q_0 = _mm_add_epi64(v_acc_q_0, v_acc_q_1);
 
 #if ARCH_X86_64
-  acc = (uint64_t)_mm_extract_epi64(v_acc_q_0, 0);
+  acc = _mm_extract_epi64(v_acc_q_0, 0);
 #else
   xx_storel_64(&acc, v_acc_q_0);
 #endif

av1/encoder/x86/wedge_utils_sse2.c

diff --git a/av1/encoder/x86/wedge_utils_sse2.c b/av1/encoder/x86/wedge_utils_sse2.c
index 6a3ef00..e665b2e 100644
--- a/av1/encoder/x86/wedge_utils_sse2.c
+++ b/av1/encoder/x86/wedge_utils_sse2.c
@@ -175,7 +175,7 @@
   v_acc_q = _mm_add_epi64(v_acc_q, _mm_srli_si128(v_acc_q, 8));
 
 #if ARCH_X86_64
-  acc = (uint64_t)_mm_cvtsi128_si64(v_acc_q);
+  acc = _mm_cvtsi128_si64(v_acc_q);
 #else
   xx_storel_64(&acc, v_acc_q);
 #endif