[MPlayer-dev-eng] [PATCH] 'Warning: missing operand; zero assumed' fixes

[Carl Eugen Hoyos]

On 2007-01-26 12:41, Dmitry Antipov wrote:
> Hello again,
> 
> This patch tries to fix all warning messages like the following
> (note {standard input} due to '-pipe'):
> 
> {standard input}: Assembler messages:
> {standard input}:13: Warning: missing operand; zero assumed
> 
> With as from binutils 2.17.50 and gcc 4.1.1, the only code where
> this takes place is mp3lib/dct64_MMX.c; Intel C reports a few more.

Smaller patch attached, only removes gcc warnings, produces binary identical 
code.

Carl Eugen
-------------- next part --------------
Index: mp3lib/dct64_3dnow.c
===================================================================
--- mp3lib/dct64_3dnow.c	(Revision 22092)
+++ mp3lib/dct64_3dnow.c	(Arbeitskopie)
@@ -21,7 +21,7 @@
     __asm __volatile(
 "	movl %2,%%eax\n\t"
 
-"	leal 128+%3,%%edx\n\t"
+"	leal (128)%3,%%edx\n\t"
 "	movl %0,%%esi\n\t"
 "	movl %1,%%edi\n\t"
 "	movl $"MANGLE(costab_mmx)",%%ebx\n\t"
Index: mp3lib/dct64_k7.c
===================================================================
--- mp3lib/dct64_k7.c	(Revision 22092)
+++ mp3lib/dct64_k7.c	(Arbeitskopie)
@@ -21,7 +21,7 @@
     __asm __volatile(
 "	movl %2,%%eax\n\t"
 
-"	leal 128+%3,%%edx\n\t"
+"	leal (128)%3,%%edx\n\t"
 "	movl %0,%%esi\n\t"
 "	movl %1,%%edi\n\t"
 "	movl $"MANGLE(costab_mmx)",%%ebx\n\t"