[FFmpeg-devel] [PATCH] AAC: unroll parts of decode_spectrum_and_dequant()

Robert Swain robert.swain
Tue Dec 9 15:01:26 CET 2008


2008/12/9 Michael Niedermayer <michaelni at gmx.at>:
> On Mon, Dec 08, 2008 at 10:34:55PM -0500, Alex Converse wrote:
>> that I also tried reverting access back to forward order and got a slight
>> speed up. This way made the unsigned loop just like the other three, so I
>> added that one for another benchmarked verified speed up.
>>
>> The net gain is a 12% decrease in cycles over the original vs 5% before.
>
> very nice, i like the patch, but i see robert is back here and reviewing
> patches so i leave things to him ...

I tend to prefer an OK from you as my programming experience is much
less than yours but, if you're happy for me to commit things when I'm
happy with the code then I shall.

Regards,
Rob




More information about the ffmpeg-devel mailing list