[FFmpeg-devel] [PATCH] AAC: use table for cbrtf(n)*n
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.
MP3 is also fixed-point, so it's probably quite different. I haven't
looked at it in detail.
mans at mansr.com
More information about the ffmpeg-devel