On Tue, May 10, 2011 at 7:16 PM, Ronald S. Bultje <[email protected]> wrote: > ---------- Forwarded message ---------- > From: Ronald S. Bultje <[email protected]> > Date: Tue, May 10, 2011 at 9:32 PM > Subject: Re: [libav-devel] [PATCH 1/2] Update H.264 deblock asm > To: libav development <[email protected]> > > > Hi, > > On Tue, May 10, 2011 at 12:14 PM, Jason Garrett-Glaser <[email protected]> wrote: >> Includes AVX versions from x264. >> --- >> libavcodec/x86/h264_deblock.asm | 395 >> ++++++++++++++++++++++----------------- >> libavcodec/x86/h264dsp_mmx.c | 61 ++++--- >> libavcodec/x86/x86util.asm | 19 ++- >> 3 files changed, 276 insertions(+), 199 deletions(-) > [..] >> - if (mm_flags&AV_CPU_FLAG_SSSE3) { >> - c->biweight_h264_pixels_tab[0]= >> ff_h264_biweight_16x16_ssse3; >> - c->biweight_h264_pixels_tab[1]= ff_h264_biweight_16x8_ssse3; >> - c->biweight_h264_pixels_tab[2]= ff_h264_biweight_8x16_ssse3; >> - c->biweight_h264_pixels_tab[3]= ff_h264_biweight_8x8_ssse3; >> - c->biweight_h264_pixels_tab[4]= ff_h264_biweight_8x4_ssse3; > > I don't think that was intended. With that reverted, the rest looks > good, I trust that x264 gives this sufficient speed-testing so I don't > have to do that myself. :-). > > (Now let me review 2/2 and then I'll push both.) > > Ronald >
Applied with fix. Jason _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
