[FFmpeg-devel] [PATCH] Port x264 SSE2 deblocking code to H.264 decoder

[Måns Rullgård]

"Jason Garrett-Glaser" <darkshikari at gmail.com> writes:

> $subject.
>
> Dark Shikari
>
> @@ -2819,6 +2821,10 @@
>              H264_QPEL_FUNCS(3, 1, sse2);
>              H264_QPEL_FUNCS(3, 2, sse2);
>              H264_QPEL_FUNCS(3, 3, sse2);
> +#if defined(CONFIG_GPL) && defined(HAVE_YASM)
> +            c->h264_v_loop_filter_luma= h264_v_loop_filter_luma_sse2;
> +            c->h264_h_loop_filter_luma= h264_h_loop_filter_luma_sse2;
> +#endif
>          }
>  #ifdef HAVE_SSSE3
>          if(mm_flags & FF_MM_SSSE3){
> Index: libavcodec/Makefile
> ===================================================================
> --- libavcodec/Makefile	(revision 16170)
> +++ libavcodec/Makefile	(working copy)
> @@ -404,7 +404,8 @@
>  MMX-OBJS-$(CONFIG_VP6A_DECODER)        += i386/vp3dsp_mmx.o i386/vp3dsp_sse2.o
>  MMX-OBJS-$(CONFIG_VP6F_DECODER)        += i386/vp3dsp_mmx.o i386/vp3dsp_sse2.o
>  MMX-OBJS-$(CONFIG_WMV3_DECODER)        += i386/vc1dsp_mmx.o
> -MMX-OBJS-$(HAVE_YASM)                  += i386/dsputil_yasm.o
> +MMX-OBJS-$(HAVE_YASM)                  += i386/dsputil_yasm.o \
> +                                          i386/h264_deblock_sse2.o
>  
>  OBJS-$(HAVE_MMX)                       += i386/cpuid.o                  \
>                                            i386/dnxhd_mmx.o              \

This could be improved by using config variable with the proper
dependencies.

Something like this in configure:

h264_deblock_sse2_deps="gpl yasm"
h264_decoder_suggest="h264_deblock_sse2"

-- 
M?ns Rullg?rd
mans at mansr.com