From f8b360c4192e0865b31a8eb589753321d4abd920 Mon Sep 17 00:00:00 2001 From: James Zern Date: Thu, 17 Apr 2025 12:21:34 -0700 Subject: [PATCH] alpha_processing_sse2: quiet signed conv warning After: 44f91b0d Speed DispatchAlpha_SSE2 up _mm_set1_epi8 takes a char argument; add a `char` cast for 0xff. from clang-14 integer sanitizer: implicit conversion from type 'int' of value 255 (32-bit, signed) to type 'char' changed the value to -1 (8-bit, signed) Change-Id: I0f4ed092eddc0beb311f44bf3d4b74a4d1177040 --- src/dsp/alpha_processing_sse2.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/dsp/alpha_processing_sse2.c b/src/dsp/alpha_processing_sse2.c index bfe40c30..7abdc737 100644 --- a/src/dsp/alpha_processing_sse2.c +++ b/src/dsp/alpha_processing_sse2.c @@ -29,7 +29,7 @@ static int DispatchAlpha_SSE2(const uint8_t* WEBP_RESTRICT alpha, int i, j; const __m128i zero = _mm_setzero_si128(); const __m128i alpha_mask = _mm_set1_epi32((int)0xff); // to preserve A - const __m128i all_0xff = _mm_set1_epi8(0xff); + const __m128i all_0xff = _mm_set1_epi8((char)0xff); __m128i all_alphas16 = all_0xff; __m128i all_alphas8 = all_0xff;