[FFmpeg-devel] [PATCH] use 'l' asm suffix to make icc happy
Michael Niedermayer
michaelni
Sun Dec 16 01:25:34 CET 2007
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>
More information about the ffmpeg-devel
mailing list