On Sun, Jul 01, 2012 at 02:56:52PM +0100, Mans Rullgard wrote: > > --- a/libavfilter/vf_gradfun.c > +++ b/libavfilter/vf_gradfun.c > @@ -135,12 +134,8 @@ static av_cold int init(AVFilterContext *ctx, const char > *args) > > - if (HAVE_MMX && cpu_flags & AV_CPU_FLAG_MMX2) > - gf->filter_line = ff_gradfun_filter_line_mmx2; > - if (HAVE_SSSE3 && cpu_flags & AV_CPU_FLAG_SSSE3) > - gf->filter_line = ff_gradfun_filter_line_ssse3; > - if (HAVE_SSE && cpu_flags & AV_CPU_FLAG_SSE2) > - gf->blur_line = ff_gradfun_blur_line_sse2; > + if (HAVE_MMX) > + ff_gradfun_init_x86(gf);
Why not check for ARCH_X86 instead of HAVE_MMX and let the arch-dependent code figure out such implementation details? Diego _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
