From cd93bf429cd7bac00e2bf92433beb1a0a4d78310 Mon Sep 17 00:00:00 2001 From: Luc Trudeau Date: Mon, 19 Feb 2018 10:16:56 -0500 Subject: [PATCH] Fix warning in idct_intrin_sse2.c In av1_iht8x4_32_add_sse2, iidtx8 was using uninitialized values for rows 4,5,6,7. Change-Id: I12b41cedde6cdcc4c153a375571853a10d26c2df --- av1/common/x86/idct_intrin_sse2.c | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/av1/common/x86/idct_intrin_sse2.c b/av1/common/x86/idct_intrin_sse2.c index 9d4a43570..434189f1d 100644 --- a/av1/common/x86/idct_intrin_sse2.c +++ b/av1/common/x86/idct_intrin_sse2.c @@ -661,6 +661,13 @@ static INLINE void write_buffer_8x4_round5(uint8_t *dest, __m128i *in, RECON_AND_STORE(dest + 3 * stride, in[3]); } +static INLINE void iidtx8x4_sse2(__m128i *in) { + in[0] = _mm_slli_epi16(in[0], 1); + in[1] = _mm_slli_epi16(in[1], 1); + in[2] = _mm_slli_epi16(in[2], 1); + in[3] = _mm_slli_epi16(in[3], 1); +} + void av1_iht8x4_32_add_sse2(const tran_low_t *input, uint8_t *dest, int stride, const TxfmParam *txfm_param) { __m128i in[8]; @@ -689,7 +696,7 @@ void av1_iht8x4_32_add_sse2(const tran_low_t *input, uint8_t *dest, int stride, case V_ADST: case V_DCT: case IDTX: - iidtx8_sse2(in); + iidtx8x4_sse2(in); array_transpose_8x8(in, in); break; default: assert(0); break; -- GitLab