On Mon, Dec 19, 2011 at 12:52:47AM +0000, Måns Rullgård wrote: > Diego Biurrun <di...@biurrun.de> writes: > > > > --- a/configure > > +++ b/configure > > @@ -939,6 +939,7 @@ CONFIG_LIST=" > > h264dsp > > h264pred > > + h264qpel > > hardcoded_tables > > @@ -1301,7 +1302,7 @@ h263_encoder_select="aandct" > > h263p_encoder_select="h263_encoder" > > -h264_decoder_select="golomb h264chroma h264dsp h264pred" > > +h264_decoder_select="golomb h264chroma h264dsp h264pred h264qpel" > > h264_dxva2_hwaccel_deps="dxva2api_h" > > [...] > > > --- a/libavcodec/x86/Makefile > > +++ b/libavcodec/x86/Makefile > > @@ -23,6 +23,7 @@ YASM-OBJS-$(CONFIG_H264DSP) += > > x86/h264_deblock.o \ > > YASM-OBJS-$(CONFIG_H264PRED) += x86/h264_intrapred.o \ > > x86/h264_intrapred_10bit.o > > MMX-OBJS-$(CONFIG_H264PRED) += x86/h264_intrapred_init.o > > +YASM-OBJS-$(CONFIG_H264QPEL) += x86/h264_qpel_10bit.o > > > > @@ -52,7 +53,6 @@ MMX-OBJS-$(CONFIG_VP8_DECODER) += > > x86/vp8dsp-init.o > > MMX-OBJS-$(HAVE_YASM) += x86/dsputil_yasm.o \ > > x86/deinterlace.o \ > > x86/fmtconvert.o \ > > - x86/h264_qpel_10bit.o \ > > $(YASM-OBJS-yes) > > > > Is the plan to make all the qpel code conditional on this?
Yes, sort of - I looked into the hairy ball of spaghetti that is dsputil_mmx.c and decided to leave splitting the QPEL code out of it for another day. But the 10bit code is trivial to split, so I did it right away. Diego _______________________________________________ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel
