Bug 1684463 - [devtools] Part 1: Shorten the _createAttribute function by refactoring...
[gecko.git] / media / libspeex_resampler / remove-empty-asm-clobber.patch
blobebb6d669833b3df7726def8c6d24f0b8efb86ad4
1 https://gcc.gnu.org/onlinedocs/gcc/Extended-Asm.html#Extended-Asm says
3 asm [volatile] ( AssemblerTemplate : [OutputOperands] [ : [InputOperands] [ : [Clobbers] ] ] )
5 which implies that Clobbers is optional even after the third colon, but
6 the gcc used for b2g_try_emulator_dep builds says
8 resample_neon.c: In function 'saturate_32bit_to_16bit':
9 resample_neon.c:50: error: expected string literal before ')' token
11 diff --git a/media/libspeex_resampler/src/resample_neon.c b/media/libspeex_resampler/src/resample_neon.c
12 --- a/media/libspeex_resampler/src/resample_neon.c
13 +++ b/media/libspeex_resampler/src/resample_neon.c
14 @@ -41,18 +41,17 @@
15 #include <arm_neon.h>
17 #ifdef FIXED_POINT
18 #ifdef __thumb2__
19 static inline int32_t saturate_32bit_to_16bit(int32_t a) {
20 int32_t ret;
21 asm ("ssat %[ret], #16, %[a]"
22 : [ret] "=&r" (ret)
23 - : [a] "r" (a)
24 - : );
25 + : [a] "r" (a));
26 return ret;
28 #else
29 static inline int32_t saturate_32bit_to_16bit(int32_t a) {
30 int32_t ret;
31 asm ("vmov.s32 d0[0], %[a]\n"
32 "vqmovn.s32 d0, q0\n"
33 "vmov.s16 %[ret], d0[0]\n"