[FFmpeg-devel] [PATCH] fix for roundup issue 2127

Ronald S. Bultje rsbultje
Fri Dec 31 17:54:15 CET 2010


Hi,

On Fri, Dec 31, 2010 at 11:30 AM, Reimar D?ffinger
<Reimar.Doeffinger at gmx.de> wrote:
> On Fri, Dec 31, 2010 at 10:59:01AM -0500, Ronald S. Bultje wrote:
>> On Fri, Dec 31, 2010 at 10:55 AM, Reimar D?ffinger
>> <Reimar.Doeffinger at gmx.de> wrote:
>> > which I think makes this unacceptable, at least if we still have to
>> > policy that fast code in as many cases as possible is more important
>> > that compilation with options that are only useful for debugging.
>>
>> The code is faster after his patch; the original code does 6 leas, the
>> new code uses direct addressing.
>
> Well, it still needs gcc to generate the *3 value. By moving that into
> the asm code (and making the strides inout values) I think it should be
> possible to make it work with only 4 registers, though it might be
> slower (instruction scheduling may be a bit problematic there).

Worth testing. :-).

Ronald



More information about the ffmpeg-devel mailing list