Re: [FFmpeg-devel] [PATCH] vp9: add fullpel (avg) SIMD for 10/12bpp.

2015-09-16 Thread Ronald S. Bultje
Hi, On Tue, Sep 15, 2015 at 9:38 PM, James Almer wrote: > On 9/15/2015 9:24 PM, Ronald S. Bultje wrote: > > --- > > libavcodec/x86/vp9dsp_init_16bpp.c | 42 > ++ > > Why not just add all this to vp9dsp_init.c and selectively initialize > everything by checking

Re: [FFmpeg-devel] [PATCH] vp9: add fullpel (avg) SIMD for 10/12bpp.

2015-09-15 Thread James Almer
On 9/15/2015 9:24 PM, Ronald S. Bultje wrote: > --- > libavcodec/x86/vp9dsp_init_16bpp.c | 42 > ++ Why not just add all this to vp9dsp_init.c and selectively initialize everything by checking the existing bpp argument in ff_vp9dsp_init_x86()? If you think you

[FFmpeg-devel] [PATCH] vp9: add fullpel (avg) SIMD for 10/12bpp.

2015-09-15 Thread Ronald S. Bultje
--- libavcodec/x86/vp9dsp_init_16bpp.c | 42 ++ libavcodec/x86/vp9mc.asm | 24 ++ 2 files changed, 58 insertions(+), 8 deletions(-) diff --git a/libavcodec/x86/vp9dsp_init_16bpp.c b/libavcodec/x86/vp9dsp_init_16bpp.c index 3319012