[FFmpeg-devel] [RFC] remove lrintf fallback implementation

Måns Rullgård mans
Sun Oct 21 03:18:45 CEST 2007


"V?ctor Paesa" <wzrlpy at arsystel.com> writes:

>>> +    asm
>>> +    ("fistpll %0"
>>> +    : "=m" (llrintres) : "t" (x) : "st");
>>
>> The formatting of this asm() statement is incredibly ugly.
>
> I must confess I am an asm-illiterate, for me every asm is ugly. I made
> another (mostly blind) attempt.

If you don't know assembler, how do you know this code is correct?

> +    asm(
> +        "fistpll %0" : "=m" (llrintres) : "t" (x) : "st"
> +    );

I, and I think most others, prefer writing it like this:

    asm("fistpll %0" : "=m" (llrintres) : "t" (x) : "st");

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




More information about the ffmpeg-devel mailing list