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

Måns Rullgård mans
Sun Dec 16 00:51:46 CET 2007


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?

> so i wouldnt be surprised if it just behaves similar for memory instead of
> registers ...

And do what?  With a memory operand, the size cannot be deduced as it
can with a register.

-- 
M?ns Rullg?rd
mans at mansr.com




More information about the ffmpeg-devel mailing list