[FFmpeg-devel] [PATCH + RFC] Faster ff_celp_lp_synthesis_filterf() (and failed SSE SIMD version)
Michael Niedermayer
michaelni
Tue Dec 15 04:21:28 CET 2009
On Mon, Dec 14, 2009 at 10:21:47PM +0100, Vitor Sessak wrote:
> Vitor Sessak wrote:
>> Michael Niedermayer wrote:
>>> On Sun, Dec 13, 2009 at 08:55:08PM +0100, Vitor Sessak wrote:
>>> [...]
>>>> + old_out3 = old_out2;
>>>> + old_out2 = old_out1;
>>>> + old_out1 = old_out0;
>>>> + old_out0 = out[-i-1];
>>>> +
>>>> + val = filter_coeffs[i];
>>>> +
>>>> + out0 -= val * old_out0;
>>>> + out1 -= val * old_out1;
>>>> + out2 -= val * old_out2;
>>>> + out3 -= val * old_out3;
>>>
>>> old_out3 = out[-i-1];
>>>
>>> val = filter_coeffs[i];
>>> out0 -= val * old_out3;
>>> out1 -= val * old_out0;
>>> out2 -= val * old_out1;
>>> out3 -= val * old_out2;
>>>
>>> and similarly you can get rid of the other copies if you unroll it more
>> Indeed, done. New patch attached.
>> BTW, in my SSE code, there was a line of code missing:
>>> DECLARE_ASM_CONST(16, uint32_t, mask[4]) = {0xFFFFFFFF, 0xFFFFFFFF,
>>> 0xFFFFFFFF, 0x00000000};
>>>
>
> Err, this time without reinventing FFSWAP()...
>
> -Vitor
do you want to be maintainer of celp_filters*
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
It is dangerous to be right in matters on which the established authorities
are wrong. -- Voltaire
-------------- 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/20091215/f74b2267/attachment.pgp>
More information about the ffmpeg-devel
mailing list