summaryrefslogtreecommitdiff
path: root/vpx_dsp
diff options
context:
space:
mode:
authorJames Zern <jzern@google.com>2016-12-19 10:51:59 -0800
committerJames Zern <jzern@google.com>2016-12-19 12:49:44 -0800
commita68b36c752efbff1ed2ed7e0d037f30489999916 (patch)
tree8c4a28d32d42535165bbb19d546e9a7da40a7ae7 /vpx_dsp
parent7e23f895ca868a5b3ad5ec8eda3433a19a58d750 (diff)
downloadlibvpx-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.c11
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;
}
// -----------------------------------------