On Thu, Jun 02, 2011 at 08:34:36PM -0700, Ronald S. Bultje wrote:
> ---
> libswscale/ppc/Makefile | 3 +-
> libswscale/ppc/swscale_altivec.c | 419
> +++++++++++++++++++++++++++++
> libswscale/ppc/swscale_altivec_template.c | 409 ----------------------------
> libswscale/ppc/yuv2rgb_altivec.c | 1 +
> libswscale/ppc/yuv2rgb_altivec.h | 34 +++
> libswscale/swscale.c | 12 +-
> libswscale/swscale_internal.h | 7 +-
> 7 files changed, 460 insertions(+), 425 deletions(-)
> create mode 100644 libswscale/ppc/swscale_altivec.c
> delete mode 100644 libswscale/ppc/swscale_altivec_template.c
> create mode 100644 libswscale/ppc/yuv2rgb_altivec.h
>
> diff --git a/libswscale/ppc/Makefile b/libswscale/ppc/Makefile
> index dd9047c..2c5b4f3 100644
> --- a/libswscale/ppc/Makefile
> +++ b/libswscale/ppc/Makefile
> @@ -1,2 +1,3 @@
> -OBJS-$(HAVE_ALTIVEC) += ppc/yuv2rgb_altivec.o \
> +OBJS-$(HAVE_ALTIVEC) += ppc/swscale_altivec.o \
> + ppc/yuv2rgb_altivec.o \
> ppc/yuv2yuv_altivec.o
> diff --git a/libswscale/ppc/swscale_altivec.c
> b/libswscale/ppc/swscale_altivec.c
> new file mode 100644
> index 0000000..acfdc94
> --- /dev/null
> +++ b/libswscale/ppc/swscale_altivec.c
[...]
> +
> +void ff_sws_init_swScale_altivec(SwsContext *c)
> +{
> + if (!(av_get_cpu_flags() & AV_CPU_FLAG_ALTIVEC))
> + return;
> +
> + c->yuv2yuvX = yuv2yuvX_altivec_real;
maybe _real suffix can be dropped now too?
> + /* The following list of supported dstFormat values should
> + * match what's found in the body of ff_yuv2packedX_altivec() */
> + if (!(c->flags & SWS_BITEXACT) && !c->alpPixBuf &&
> + (c->dstFormat==PIX_FMT_ABGR || c->dstFormat==PIX_FMT_BGRA ||
> + c->dstFormat==PIX_FMT_BGR24 || c->dstFormat==PIX_FMT_RGB24 ||
> + c->dstFormat==PIX_FMT_RGBA || c->dstFormat==PIX_FMT_ARGB)) {
> + c->yuv2packedX = ff_yuv2packedX_altivec;
> + }
too much indentation for the closing brace
otherwise looks ok
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel
