[FFmpeg-devel] [PATCH] Fix PPC build.

Måns Rullgård mans
Fri Feb 18 02:02:05 CET 2011


"Ronald S. Bultje" <rsbultje at gmail.com> writes:

> ---
>  libavcodec/ppc/h264_altivec.c          |    6 ------
>  libavcodec/ppc/h264_template_altivec.c |   10 ++++++++++
>  libavcodec/ppc/vc1dsp_altivec.c        |   17 +++++++++++++++++
>  3 files changed, 27 insertions(+), 6 deletions(-)
>

[...]

> diff --git a/libavcodec/ppc/h264_template_altivec.c b/libavcodec/ppc/h264_template_altivec.c
> index 8cf39c8..29e4d40 100644
> --- a/libavcodec/ppc/h264_template_altivec.c
> +++ b/libavcodec/ppc/h264_template_altivec.c
> @@ -75,6 +75,7 @@
>  #define noop(a) a
>  #define add28(a) vec_add(v28ss, a)
>
> +#ifdef PREFIX_h264_chroma_mc8_altivec
>  static void PREFIX_h264_chroma_mc8_altivec(uint8_t * dst, uint8_t * src,
>                                      int stride, int h, int x, int y) {
>      DECLARE_ALIGNED(16, signed int, ABCD)[4] =
> @@ -201,8 +202,10 @@ static void PREFIX_h264_chroma_mc8_altivec(uint8_t * dst, uint8_t * src,
>          }
>      }
>  }
> +#endif

I'd prefer not adding to the already excessive ifdeffery in this file.
Is there a reason you didn't move the vc1 functions to a different file
instead?

-- 
M?ns Rullg?rd
mans at mansr.com



More information about the ffmpeg-devel mailing list