diff options
author | James Zern <jzern@google.com> | 2016-12-19 10:51:59 -0800 |
---|---|---|
committer | James Zern <jzern@google.com> | 2016-12-19 12:49:44 -0800 |
commit | a68b36c752efbff1ed2ed7e0d037f30489999916 (patch) | |
tree | 8c4a28d32d42535165bbb19d546e9a7da40a7ae7 /vpx_dsp | |
parent | 7e23f895ca868a5b3ad5ec8eda3433a19a58d750 (diff) | |
download | libvpx-a68b36c752efbff1ed2ed7e0d037f30489999916.tar libvpx-a68b36c752efbff1ed2ed7e0d037f30489999916.tar.gz libvpx-a68b36c752efbff1ed2ed7e0d037f30489999916.tar.bz2 libvpx-a68b36c752efbff1ed2ed7e0d037f30489999916.zip |
vpx_idct32x32_1024_add_neon: quiet uninitialized warning
relocate the assignment to 'in' outside of the for loop. this quiets a
spurious warning in visual studio builds since:
86e340c enable vpx_idct32x32_1024_add_neon in hbd builds
+ give the variable a more descriptive name
BUG=webm:1294
Change-Id: I5c3da5c7939621477e0fc0ad3a1b2a3045c5bffd
Diffstat (limited to 'vpx_dsp')
-rw-r--r-- | vpx_dsp/arm/idct32x32_add_neon.c | 11 |
1 files changed, 5 insertions, 6 deletions
diff --git a/vpx_dsp/arm/idct32x32_add_neon.c b/vpx_dsp/arm/idct32x32_add_neon.c index a0b5066c6..de1bf9787 100644 --- a/vpx_dsp/arm/idct32x32_add_neon.c +++ b/vpx_dsp/arm/idct32x32_add_neon.c @@ -430,21 +430,20 @@ void vpx_idct32x32_1024_add_neon(const tran_low_t *input, uint8_t *dest, int16_t trans_buf[32 * 8]; int16_t pass1[32 * 32]; int16_t pass2[32 * 32]; - int16_t *in, *out; + const int16_t *input_pass2 = pass1; // input of pass2 is the result of pass1 + int16_t *out; int16x8_t q0s16, q1s16, q2s16, q3s16, q4s16, q5s16, q6s16, q7s16; int16x8_t q8s16, q9s16, q10s16, q11s16, q12s16, q13s16, q14s16, q15s16; for (idct32_pass_loop = 0, out = pass1; idct32_pass_loop < 2; - idct32_pass_loop++, - in = pass1, // the input of pass2 is the result of pass1 - out = pass2) { + idct32_pass_loop++, out = pass2) { for (i = 0; i < 4; i++, out += 8) { // idct32_bands_loop if (idct32_pass_loop == 0) { idct32_transpose_pair_tran_low(input, trans_buf); input += 32 * 8; } else { - idct32_transpose_pair(in, trans_buf); - in += 32 * 8; + idct32_transpose_pair(input_pass2, trans_buf); + input_pass2 += 32 * 8; } // ----------------------------------------- |