[FFmpeg-devel] [PATCH]HE-AACv1 try 3 (all missing functionality added)

[Michael Niedermayer]

On Sat, Feb 13, 2010 at 05:31:59PM +0100, Vitor Sessak wrote:
[...]
>> +
>> +static void vector_fmul_scalar_fadd_scalar_c(float *dst, const float 
>> *src,
>> +                                             float mul, float add, int 
>> len)
>> +{
>> +    int i;
>> +    for (i = 0; i < len; i++)
>> +        dst[i] = src[i] * mul + add;
>> +}
>> +
>
> Did you benchmark if hand-unrolling these loops give any benefit? You know 
> that len is a multiple of four, but the compiler don't.

assert(len>0 && len%4==0);
then the compiler knows it too :)
and after you looked at what gcc generates if forced to unroll
(theres a flag for that IIRC)
then post a code pessimization bug to the gcc-devels
-> only way to see gcc get better at these things

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Frequently ignored awnser#1 FFmpeg bugs should be sent to our bugtracker. User
questions about the command line tools should be sent to the ffmpeg-user ML.
And questions about how to use libav* should be sent to the libav-user ML.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100213/18c8399e/attachment.pgp>