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

Reply via email to