[FFmpeg-devel] [PATCH] Port AAC decoder to use ff_imdct_half and vector_fmul_window

Steven Ellis mail_lists
Sun Aug 24 01:07:24 CEST 2008


On 23/08/2008, at 6:18 AM, Robert Swain wrote:

> 2008/8/22 Michael Niedermayer <michaelni at gmx.at>:
>> On Fri, Aug 22, 2008 at 03:03:44PM +0100, Robert Swain wrote:
>>> Attached are:
>>> 0001-Separate-IMDCT-and-windowing-into-imdct-windowing-a.patch
>>> I thought the current layout of the code was quite difficult to look
>>> at as it mixes imdct, windowing and buffer updates. I decided
>>> separating them into clearer sections made the code more easy to
>>> understand and think about.
>>>
>>> 0002-Whitespace-cosmetics.patch
>>> This cleans up alignment and such after 0001.
>>>
>>> 0003-Use-imdct_half-and-fmul_window.-Reduce-buffer-sizes.patch
>>> Does exactly what it says on the tin. :) I observe an ~10%  
>>> performance
>>> improvement after these patches.
>>>
>>> Comments welcome as always. Benchmarks are below for this file.
>>
>> looks ok
>
> Applied. Did you note the removal of the revers array and the other
> array size reductions? Also the memset() and unnecessary addition of
> 0s has gone. :)
>
>> btw, i think we should drop faad support ...
>
> I'd like to leave it for the moment. As it has features we don't, it's
> useful for testing.
>

Rob - Does your implementation cover full SBR support for AAC-LC and  
HE-AACv2?

Steve




More information about the ffmpeg-devel mailing list