Re: [FFmpeg-devel] [PATCH] vp9: add avx2 iadst16 implementations.

2016-11-15 Thread Ronald S. Bultje
Hi, On Mon, Nov 14, 2016 at 4:26 PM, James Almer wrote: > On 11/8/2016 1:22 PM, Ronald S. Bultje wrote: > > Also a small cosmetic change to the avx2 idct16 version to make it > > explicit that one of the arguments to the write-out macros is unused > > for >=avx2 (it uses pmovzxbw instead of punp

Re: [FFmpeg-devel] [PATCH] vp9: add avx2 iadst16 implementations.

2016-11-14 Thread James Almer
On 11/8/2016 1:22 PM, Ronald S. Bultje wrote: > Also a small cosmetic change to the avx2 idct16 version to make it > explicit that one of the arguments to the write-out macros is unused > for >=avx2 (it uses pmovzxbw instead of punpcklbw). A braindead test (ffmpeg -i 4kHDRsample.webm -benchmark -f

Re: [FFmpeg-devel] [PATCH] vp9: add avx2 iadst16 implementations.

2016-11-08 Thread Ronald S. Bultje
Hi, On Tue, Nov 8, 2016 at 6:33 PM, compn wrote: > On Tue, 8 Nov 2016 14:44:32 -0500 > "Ronald S. Bultje" wrote: > > > >> x86-32 linux gets stomach ache from this: > > >> > > >> YASMlibavcodec/x86/vp9itxfm.o > > >> src/libavcodec/x86/vp9itxfm.asm:2150: error: instruction expected > > >> aft

Re: [FFmpeg-devel] [PATCH] vp9: add avx2 iadst16 implementations.

2016-11-08 Thread compn
On Tue, 8 Nov 2016 14:44:32 -0500 "Ronald S. Bultje" wrote: > >> x86-32 linux gets stomach ache from this: > >> > >> YASMlibavcodec/x86/vp9itxfm.o > >> src/libavcodec/x86/vp9itxfm.asm:2150: error: instruction expected > >> after label > >> src/libavcodec/x86/vp9itxfm.asm:2151: error: instruct

Re: [FFmpeg-devel] [PATCH] vp9: add avx2 iadst16 implementations.

2016-11-08 Thread Ronald S. Bultje
Hi, On Tue, Nov 8, 2016 at 2:43 PM, Ronald S. Bultje wrote: > Hi, > > On Tue, Nov 8, 2016 at 2:39 PM, Michael Niedermayer < > mich...@niedermayer.cc> wrote: > >> On Tue, Nov 08, 2016 at 11:22:31AM -0500, Ronald S. Bultje wrote: >> > Also a small cosmetic change to the avx2 idct16 version to make

Re: [FFmpeg-devel] [PATCH] vp9: add avx2 iadst16 implementations.

2016-11-08 Thread Ronald S. Bultje
Hi, On Tue, Nov 8, 2016 at 2:39 PM, Michael Niedermayer wrote: > On Tue, Nov 08, 2016 at 11:22:31AM -0500, Ronald S. Bultje wrote: > > Also a small cosmetic change to the avx2 idct16 version to make it > > explicit that one of the arguments to the write-out macros is unused > > for >=avx2 (it us

Re: [FFmpeg-devel] [PATCH] vp9: add avx2 iadst16 implementations.

2016-11-08 Thread Michael Niedermayer
On Tue, Nov 08, 2016 at 11:22:31AM -0500, Ronald S. Bultje wrote: > Also a small cosmetic change to the avx2 idct16 version to make it > explicit that one of the arguments to the write-out macros is unused > for >=avx2 (it uses pmovzxbw instead of punpcklbw). > --- > libavcodec/x86/vp9dsp_init.c |

[FFmpeg-devel] [PATCH] vp9: add avx2 iadst16 implementations.

2016-11-08 Thread Ronald S. Bultje
Also a small cosmetic change to the avx2 idct16 version to make it explicit that one of the arguments to the write-out macros is unused for >=avx2 (it uses pmovzxbw instead of punpcklbw). --- libavcodec/x86/vp9dsp_init.c | 8 +- libavcodec/x86/vp9itxfm.asm | 188