[FFmpeg-devel] [PATCH] use 'l' asm suffix to make icc happy

[Michael Niedermayer]

On Sat, Dec 15, 2007 at 11:51:46PM +0000, M?ns Rullg?rd wrote:
> Michael Niedermayer <michaelni at gmx.at> writes:
> 
> > Hi
> >
> > On Sat, Dec 15, 2007 at 11:06:32PM +0100, Reimar D?ffinger wrote:
> >> Hello,
> >> On Sat, Dec 15, 2007 at 09:56:20PM +0000, M?ns Rullg?rd wrote:
> >> [...]
> >> > With what little x86 assembler I know, that looks OK (and it builds
> >> > over here too).  However, there are several more similar instances in
> >> > the same file.  I suppose they happened to get compiled differently,
> >> > and thus didn't cause you any trouble.
> >> 
> >> attached patch should fix the rest as well.
> >> I wonder why it causes no problem with gcc, does it always load it into
> >> a register?
> >
> > the gnu assembler does not do any error checking, at least not when i last
> > tried a few years ago ... maybe it has been fixed ...
> > but in the past it happily compiled code like
> > decl %al
> 
> What machine instruction did it generate?

i do not remember (it was long ago)i dont even remember if it was dec
i just remember gas ignored the inconsistancy silently and generated a
valid instruction. i also dont know if it ignored the postfix or register
size

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

I have never wished to cater to the crowd; for what I know they do not
approve, and what they approve I do not know. -- Epicurus
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20071216/43dae33f/attachment.pgp>