[FFmpeg-devel] [PATCH] use 'l' asm suffix to make icc happy
Sun Dec 16 00:51:46 CET 2007
Michael Niedermayer <michaelni at gmx.at> writes:
> On Sat, Dec 15, 2007 at 11:06:32PM +0100, Reimar D?ffinger wrote:
>> 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.
mans at mansr.com
More information about the ffmpeg-devel