commit 9145202c61433682ba26aeb283c87d774f41032a
author Andy Hung <hunga@google.com> Thu Jan 05 17:11:19 2017 -0800
committer Andy Hung <hunga@google.com> Fri Jan 06 16:37:42 2017 -0800
tree 2a15888e8dbc7e370f2bab33847cafeb0e7dd7f0
parent 51157baa6c8b142ec2c723ffd0cf8dc53565ca11

AudioResampler: Workaround for Clang assembly bug

Used to work on M with gcc.

Test: Native audioresampler tests on Pixel XL disabling NEON
Bug: 34110890
Change-Id: Ibaaaf9ed833a37a9017a3217d31cb5ced6fa2c82

services/audioflinger/AudioResamplerFirOps.h

diff --git a/services/audioflinger/AudioResamplerFirOps.h b/services/audioflinger/AudioResamplerFirOps.h
index 2a26496..5dc1790 100644
--- a/services/audioflinger/AudioResamplerFirOps.h
+++ b/services/audioflinger/AudioResamplerFirOps.h
@@ -119,7 +119,7 @@
 static inline
 int32_t mulAddRL(int left, uint32_t inRL, int16_t v, int32_t a)
 {
-#if USE_INLINE_ASSEMBLY
+#if 0 // USE_INLINE_ASSEMBLY Seems to fail with Clang b/34110890
     int32_t out;
     if (left) {
         asm( "smlabb %[out], %[v], %[inRL], %[a] \n"
@@ -142,7 +142,7 @@
 static inline
 int32_t mulAddRL(int left, uint32_t inRL, int32_t v, int32_t a)
 {
-#if USE_INLINE_ASSEMBLY
+#if 0 // USE_INLINE_ASSEMBLY Seems to fail with Clang b/34110890
     int32_t out;
     if (left) {
         asm( "smlawb %[out], %[v], %[inRL], %[a] \n"