From 2951a6df4a75bc711f4865ab6fe0469c04780562 Mon Sep 17 00:00:00 2001 From: "kma@webrtc.org" Date: Thu, 7 Mar 2013 18:25:34 +0000 Subject: [PATCH] Fixed an assembly code error in AECM for ARMv7. Possibly related to an AECM quality issue encountered at Chrome testing. No bug was logged. Review URL: https://webrtc-codereview.appspot.com/1160006 git-svn-id: http://webrtc.googlecode.com/svn/trunk@3631 4adac7df-926f-26a2-2b94-8c16560cd09d --- webrtc/modules/audio_processing/aecm/aecm_core.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/webrtc/modules/audio_processing/aecm/aecm_core.c b/webrtc/modules/audio_processing/aecm/aecm_core.c index 136826358..a2d43775a 100644 --- a/webrtc/modules/audio_processing/aecm/aecm_core.c +++ b/webrtc/modules/audio_processing/aecm/aecm_core.c @@ -1380,8 +1380,8 @@ static int TimeToFrequencyDomain(AecmCore_t* aecm, int i = 0; int time_signal_scaling = 0; - WebRtc_Word32 tmp32no1; - WebRtc_Word32 tmp32no2; + WebRtc_Word32 tmp32no1 = 0; + WebRtc_Word32 tmp32no2 = 0; // In fft_buf, +16 for 32-byte alignment. WebRtc_Word16 fft_buf[PART_LEN4 + 16]; @@ -1476,7 +1476,7 @@ static int TimeToFrequencyDomain(AecmCore_t* aecm, __asm __volatile( "smulbb %[tmp32no1], %[real], %[real]\n\t" "smlabb %[tmp32no2], %[imag], %[imag], %[tmp32no1]\n\t" - :[tmp32no1]"=r"(tmp32no1), + :[tmp32no1]"+r"(tmp32no1), [tmp32no2]"=r"(tmp32no2) :[real]"r"(freq_signal[i].real), [imag]"r"(freq_signal[i].imag)