[FFmpeg-devel] [PATCH] Port AAC decoder to use ff_imdct_half and vector_fmul_window
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:
>>> 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.
>>> This cleans up alignment and such after 0001.
>>> Does exactly what it says on the tin. :) I observe an ~10%
>>> 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
More information about the ffmpeg-devel