On 12/21/2012 12:41 PM, Christophe Gisquet wrote:
> 2012/12/20 Christophe Gisquet <[email protected]>:
>> 2012/12/7 Christophe Gisquet <[email protected]>:
>>> No action on my side, but does this warrant a new patch, or is there a
>>> need for another review?
>>
>> Ping2
> 
> Messed up a bit my tree, so here's a strictly identical patch, except
> for the commit message and the place where the code is moved (after
> apply_noise).

Patch ok. Probably an AVX version would benefit from putting [ps_mask2]
in a register to use 3-arg xorps with memory arg, but that can be done
later. Your SSE version is fine.

Thanks,
Justin
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to