[FFmpeg-devel] [PATCH] AAC: use table for cbrtf(n)*n

Måns Rullgård mans
Mon Jan 11 21:22:11 CET 2010


Alex Converse <alex.converse at gmail.com> writes:

> 2010/1/11 M?ns Rullg?rd <mans at mansr.com>:
>> Alex Converse <alex.converse at gmail.com> writes:
>>
>>> On Mon, Jan 11, 2010 at 8:31 AM, Mans Rullgard <mans at mansr.com> wrote:
>>>> Up to 10% faster on Cortex-A8.
>>>> ---
>>>> ?libavcodec/aac.c | ? 13 +++++++++----
>>>> ?1 files changed, 9 insertions(+), 4 deletions(-)
>>>>
>>>
>>> Otherwise LGTM
>>
>> So OK to apply this one?
>>
>
> The only other potential issue I can think of is that MP3 uses this
> same 4/3 quantizer and has it's own set of hacks to deal with it. But
> I am OK with committing this as is and leaving reconciling the two as
> a fight for another day.

Done.

MP3 is also fixed-point, so it's probably quite different.  I haven't
looked at it in detail.

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



More information about the ffmpeg-devel mailing list