diff options
author | Johann <johannkoenig@google.com> | 2019-01-07 09:53:18 -0800 |
---|---|---|
committer | Johann <johannkoenig@google.com> | 2019-01-07 09:53:18 -0800 |
commit | 6efdd9ad483f7d86e503ab16e986202feed8b105 (patch) | |
tree | 0d7da222c7ff37203e211020f311ddd1fa140524 | |
parent | b625feb3588e7e598fab2c0df1e28f2ea0a7b3e1 (diff) | |
download | libvpx-6efdd9ad483f7d86e503ab16e986202feed8b105.tar libvpx-6efdd9ad483f7d86e503ab16e986202feed8b105.tar.gz libvpx-6efdd9ad483f7d86e503ab16e986202feed8b105.tar.bz2 libvpx-6efdd9ad483f7d86e503ab16e986202feed8b105.zip |
fix vp9 fdct_quant
Values in [q]coeff1 were not correctly stored. This caused a segfault
in the sse2 libvpx__nightly_optimization jobs.
Broken in:
commit 85032bac388917916f7a149173db8b34e93e8f6e
Author: Johann <johannkoenig@google.com>
Date: Fri Dec 21 00:27:00 2018 +0000
fdct_quant: resolve missing declarations
BUG=webm:1584
Change-Id: I5f5fad34ec5e32023f5b40ff3691125754c11ced
-rw-r--r-- | vp9/encoder/x86/vp9_dct_intrin_sse2.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/vp9/encoder/x86/vp9_dct_intrin_sse2.c b/vp9/encoder/x86/vp9_dct_intrin_sse2.c index cae524e25..8426b9475 100644 --- a/vp9/encoder/x86/vp9_dct_intrin_sse2.c +++ b/vp9/encoder/x86/vp9_dct_intrin_sse2.c @@ -499,14 +499,14 @@ void vp9_fdct8x8_quant_sse2(const int16_t *input, int stride, qcoeff1 = _mm_sub_epi16(qcoeff1, coeff1_sign); store_tran_low(qcoeff0, qcoeff_ptr + n_coeffs); - store_tran_low(qcoeff0, qcoeff_ptr + n_coeffs + 8); + store_tran_low(qcoeff1, qcoeff_ptr + n_coeffs + 8); coeff0 = _mm_mullo_epi16(qcoeff0, dequant); dequant = _mm_unpackhi_epi64(dequant, dequant); coeff1 = _mm_mullo_epi16(qcoeff1, dequant); store_tran_low(coeff0, dqcoeff_ptr + n_coeffs); - store_tran_low(coeff0, dqcoeff_ptr + n_coeffs + 8); + store_tran_low(coeff1, dqcoeff_ptr + n_coeffs + 8); } { |