diff options
| author | Martin Storsjo <martin@martin.st> | 2012-01-12 14:25:29 +0200 |
|---|---|---|
| committer | Martin Storsjo <martin@martin.st> | 2012-01-13 21:20:03 +0200 |
| commit | 721f2bb7b28891c0811a5bea31fa62644131befc (patch) | |
| tree | 833966632b2ef5fbeaf77501f1b96585e145cd2c /media/libstagefright/codecs/aacenc/basic_op | |
| parent | b676a05348e4c516fa8b57e33b10548e6142c3f8 (diff) | |
| download | frameworks_av-721f2bb7b28891c0811a5bea31fa62644131befc.zip frameworks_av-721f2bb7b28891c0811a5bea31fa62644131befc.tar.gz frameworks_av-721f2bb7b28891c0811a5bea31fa62644131befc.tar.bz2 | |
stagefright aacenc: Fix an inline asm constraint
Output-only parameters should use =r or =&r constrains, never +r.
This avoids a warning about the variable 'result' being used
uninitialized.
This avoids loading the uninitialized value into the register,
before it is overwritten by the multiplication.
Change-Id: I0ef6179e133c35d290feb8e12bea180ecae11a05
Diffstat (limited to 'media/libstagefright/codecs/aacenc/basic_op')
| -rw-r--r-- | media/libstagefright/codecs/aacenc/basic_op/oper_32b.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/media/libstagefright/codecs/aacenc/basic_op/oper_32b.h b/media/libstagefright/codecs/aacenc/basic_op/oper_32b.h index 9ebd1c2..6e5844f 100644 --- a/media/libstagefright/codecs/aacenc/basic_op/oper_32b.h +++ b/media/libstagefright/codecs/aacenc/basic_op/oper_32b.h @@ -63,7 +63,7 @@ __inline Word32 L_mpy_wx(Word32 L_var2, Word16 var1) Word32 result; asm volatile( "SMULWB %[result], %[L_var2], %[var1] \n" - :[result]"+r"(result) + :[result]"=r"(result) :[L_var2]"r"(L_var2), [var1]"r"(var1) ); return result; |