[FFmpeg-devel] [PATCH] x86: hevc: adding transform_add

Michael Niedermayer michaelni at gmx.at
Wed Aug 20 01:38:09 CEST 2014


On Mon, Aug 18, 2014 at 03:28:02PM -0300, James Almer wrote:
> On 18/08/14 5:01 AM, Pierre Edouard Lepere wrote:
> > Hi, 
> > here's the new version of the patch. Sorry for the delay.
> > James, I have not done 8-bit AVX versions because it requires unpacks that are done differently in AVX.
> 
> Aren't you thinking of AVX2 with 256bits wide registers? With AVX i mean an xmm version of 
> the function that takes advantage of the three-operand instruction format, which should be 
> essentially the same as the SSE2 version except for the many saved movdqa between registers.
> 
> Anyway, LGTM and this can be committed as is. There's some of room for improvements (AVX
> version, or making all the functions use 8 or less xmm registers) but that can be done later 
> instead of sending the same patch a fourth time.
> 
> Not sure if someone else wants to comment (Ronald?).

applied

thanks

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

When the tyrant has disposed of foreign enemies by conquest or treaty, and
there is nothing more to fear from them, then he is always stirring up
some war or other, in order that the people may require a leader. -- Plato
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20140820/52173aec/attachment.asc>


More information about the ffmpeg-devel mailing list