[FFmpeg-devel] [PATCH 1/2] Rename FF_MM_ symbols related to MultiMedia CPU features as AV_CPU_MM_ symbols, as they are public, and move them from libavcodec/avcodec.h to libavutil/cpu.h.

Måns Rullgård mans
Mon Aug 30 00:59:35 CEST 2010


Stefano Sabatini <stefano.sabatini-lala at poste.it> writes:

> ---
>  libavcodec/arm/dsputil_init_arm.c |    2 +-
>  libavcodec/arm/dsputil_iwmmxt.c   |    7 ++--
>  libavcodec/arm/mpegvideo_iwmmxt.c |    3 +-
>  libavcodec/audioconvert.h         |    4 +-
>  libavcodec/avcodec.h              |   36 ++++++++++-----------
>  libavcodec/dct-test.c             |   22 ++++++------
>  libavcodec/h263dec.c              |    2 +-
>  libavcodec/motion-test.c          |    6 ++--
>  libavcodec/ppc/dsputil_ppc.c      |    2 +-
>  libavcodec/x86/cavsdsp_mmx.c      |    4 +-
>  libavcodec/x86/cpuid.c            |   52 +++++++++++++++---------------
>  libavcodec/x86/dnxhd_mmx.c        |    2 +-
>  libavcodec/x86/dsputil_mmx.c      |   65 +++++++++++++++++++------------------
>  libavcodec/x86/dsputilenc_mmx.c   |   18 +++++-----
>  libavcodec/x86/fft.c              |    8 ++--
>  libavcodec/x86/h264dsp_mmx.c      |   10 +++---
>  libavcodec/x86/motion_est_mmx.c   |    6 ++--
>  libavcodec/x86/mpegaudiodec_mmx.c |    2 +-
>  libavcodec/x86/mpegvideo_mmx.c    |   10 +++---
>  libavcodec/x86/snowdsp_mmx.c      |    7 ++--
>  libavcodec/x86/vc1dsp_mmx.c       |   12 +++---
>  libavcodec/x86/vp56dsp_init.c     |    4 +-
>  libavcodec/x86/vp8dsp-init.c      |   14 ++++----
>  libavutil/Makefile                |    1 +
>  libavutil/cpu.h                   |   44 +++++++++++++++++++++++++
>  25 files changed, 195 insertions(+), 148 deletions(-)
>  create mode 100644 libavutil/cpu.h
>
> diff --git a/libavcodec/arm/dsputil_init_arm.c b/libavcodec/arm/dsputil_init_arm.c
> index c9c3351..d1c482b 100644
> --- a/libavcodec/arm/dsputil_init_arm.c
> +++ b/libavcodec/arm/dsputil_init_arm.c
> @@ -75,7 +75,7 @@ static void simple_idct_arm_add(uint8_t *dest, int line_size, DCTELEM *block)
>
>  int mm_support(void)
>  {
> -    return HAVE_IWMMXT * FF_MM_IWMMXT;
> +    return HAVE_IWMMXT * AV_CPU_MM_IWMMXT;
>  }

Is the MM part really needed?

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



More information about the ffmpeg-devel mailing list