On 07/25/2012 11:52 PM, Ronald S. Bultje wrote: > From: "Ronald S. Bultje" <[email protected]> > > --- > libavcodec/x86/dct32_sse.asm | 2 -- > libavcodec/x86/dsputil_yasm.asm | 14 -------------- > libavcodec/x86/fft_mmx.asm | 6 ------ > libavresample/x86/audio_convert.asm | 10 ---------- > libavresample/x86/audio_mix.asm | 10 ---------- > libavutil/x86/float_dsp.asm | 10 ---------- > libavutil/x86/x86inc.asm | 15 ++++++++++++--- > 7 files changed, 12 insertions(+), 55 deletions(-) > > diff --git a/libavcodec/x86/dct32_sse.asm b/libavcodec/x86/dct32_sse.asm > index e3c8a45..351c88d 100644 > --- a/libavcodec/x86/dct32_sse.asm > +++ b/libavcodec/x86/dct32_sse.asm > @@ -278,8 +278,6 @@ cglobal dct32_float_avx, 2,3,8, out, in, tmp > vperm2f128 m0, m1, m1, 0x31 > vmovaps [outq+96], m1 > > - vzeroupper > - > ; pass 6, no SIMD... > INIT_XMM > PASS6_AND_PERMUTE
Leave that one (discussed on IRC, and see what Loren said). The rest of the patch looks good to me. Thanks, Justin _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
