On Mon, Jun 13, 2016 at 09:04:15PM +0300, Martin Storsjö wrote: > On Mon, 13 Jun 2016, Diego Biurrun wrote: > > This avoids SIMD-optimized functions having to sign-extend their > > line size argument manually to be able to do pointer arithmetic. > > --- > > > > Added missing changes for h264idct. > > > > libavcodec/aarch64/h264dsp_init_aarch64.c | 37 +++++++++++------------ > > libavcodec/arm/h264dsp_init_arm.c | 37 +++++++++++------------ > > libavcodec/h264_mb.c | 6 ++-- > > libavcodec/h264_mb_template.c | 2 +- > > libavcodec/h264addpx_template.c | 4 +-- > > libavcodec/h264dsp.h | 49 > > ++++++++++++++++--------------- > > libavcodec/h264dsp_template.c | 36 +++++++++++------------ > > libavcodec/h264idct.h | 19 ++++++------ > > libavcodec/h264idct_template.c | 30 ++++++++++++------- > > libavcodec/ppc/h264dsp.c | 31 +++++++++---------- > > libavcodec/x86/h264_deblock.asm | 20 ++++++------- > > libavcodec/x86/h264_deblock_10bit.asm | 14 ++++----- > > libavcodec/x86/h264_idct.asm | 48 > > +++++++++++++++--------------- > > libavcodec/x86/h264_idct_10bit.asm | 18 ++++++------ > > libavcodec/x86/h264_weight.asm | 4 +-- > > libavcodec/x86/h264_weight_10bit.asm | 4 +-- > > libavcodec/x86/h264dsp_init.c | 22 +++++++------- > > tests/checkasm/h264dsp.c | 4 +-- > > 18 files changed, 199 insertions(+), 186 deletions(-) > > This doesn't do the actual cleanup of the asm that I had expected.
Yes, I know. The more pressing problem, however, is that this patch fails checkasm on power7. Until that is resolved, this patch is on hold anyway. Diego _______________________________________________ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel