[FFmpeg-devel] [PATCH] Add x86-optimized versions of exponent_min().

[Måns Rullgård]

Justin Ruggles <justin.ruggles at gmail.com> writes:

> ---
>  libavcodec/Makefile         |    6 +++-
>  libavcodec/ac3dsp.c         |   51 ++++++++++++++++++++++++++++++++++++
>  libavcodec/ac3dsp.h         |   44 +++++++++++++++++++++++++++++++
>  libavcodec/ac3enc.c         |   35 +++++--------------------
>  libavcodec/x86/Makefile     |    4 +++
>  libavcodec/x86/ac3dsp.asm   |   60 +++++++++++++++++++++++++++++++++++++++++++
>  libavcodec/x86/ac3dsp_mmx.c |   47 +++++++++++++++++++++++++++++++++
>  libavcodec/x86/x86util.asm  |   10 +++++++
>  8 files changed, 227 insertions(+), 30 deletions(-)
>  create mode 100644 libavcodec/ac3dsp.c
>  create mode 100644 libavcodec/ac3dsp.h
>  create mode 100644 libavcodec/x86/ac3dsp.asm
>  create mode 100644 libavcodec/x86/ac3dsp_mmx.c
>
> On 01/28/2011 04:09 PM, Ronald S. Bultje wrote:
>> [..]
>>> > +    if (mm_flags & AV_CPU_FLAG_SSE2) {
>> HAVE_SSE2 also
>
> There is no such thing. And HAVE_SSE is handled by av_get_cpu_flags().

If configured with --disable-sse the SSE functions should not be
built, and then this needs a compile-time constant check.

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