[FFmpeg-devel] [PATCH 0/5][RFC] Speed up AAC decoder

Måns Rullgård mans
Mon Jan 11 14:45:00 CET 2010


Robert Swain <robert.swain at gmail.com> writes:

> Hello,
>
> On 11/01/10 13:37, Mans Rullgard wrote:
>> This patch series speeds up the AAC decoder by 25-50% on Cortex-A8 and
>> by 20-30% on Core i7.
>>
>> Some of the more obscure parts certainly need a few comments, which I
>> will add if there are no objections to this approach.
>
> I've looked through to see what you've done and on the whole it looks
> good to me.
>
> Personally I prefer some parentheses around not-so-well-known operator
> precedence ambiguities for clarification and some more vertical
> alignment of assignments, but that's just cosmetics.

You really should memorise the precedence and associativity of shifts,
bitwise, and arithmetic ops.  When you know the rules, paren-less code
is much easier to look at.

I'll see about the alignment.

> I would request that Alex run this code through the compliance tests
> to make sure we have no regressions, if he has time.

That would be appreciated.

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



More information about the ffmpeg-devel mailing list