stagefright aacenc: Remove useless inline asm for simple right shift

mansr · mstorsjo · commit 2ec6c483fcf5 · 2012-01-13T21:16:14.000+02:00
Change-Id: If1f40e9c16952182e974af1c86a14995259c2ade
diff --git a/media/libstagefright/codecs/aacenc/basic_op/basic_op.h b/media/libstagefright/codecs/aacenc/basic_op/basic_op.h
@@ -227,13 +227,7 @@ Word32 L_shr_r (Word32 L_var1, Word16 var2);
 #if ARMV4_INASM
 __inline Word32 ASM_L_shr(Word32 L_var1, Word16 var2)
 {
-	Word32 result;
-	asm (
-		"MOV %[result], %[L_var1], ASR %[var2] \n"
-		:[result]"=r"(result)
-		:[L_var1]"r"(L_var1), [var2]"r"(var2)
-		);
-	return result;
+	return L_var1 >> var2;
 }
 
 __inline Word32 ASM_L_shl(Word32 L_var1, Word16 var2)

Original file line number	Diff line number	Diff line change
`@@ -227,13 +227,7 @@ Word32 L_shr_r (Word32 L_var1, Word16 var2);`
`227`	`227`	`#if ARMV4_INASM`
`228`	`228`	`__inline Word32 ASM_L_shr(Word32 L_var1, Word16 var2)`
`229`	`229`	`{`
`230`		`- Word32 result;`
`231`		`- asm (`
`232`		`- "MOV %[result], %[L_var1], ASR %[var2] \n"`
`233`		`- :[result]"=r"(result)`
`234`		`- :[L_var1]"r"(L_var1), [var2]"r"(var2)`
`235`		`- );`
`236`		`- return result;`
	`230`	`+ return L_var1 >> var2;`
`237`	`231`	`}`
`238`	`232`
`239`	`233`	`__inline Word32 ASM_L_shl(Word32 L_var1, Word16 var2)`