[FFmpeg-devel] [PATCH] avcodec/x86/cavsdsp: Put MMX code under mmx check

James Almer jamrial at gmail.com
Sun Mar 5 05:00:16 EET 2017


On 3/4/2017 11:42 PM, Michael Niedermayer wrote:
> Without this the FPU state becomes trashed and causes mysterious
> fate failures with cpuflags=0
> 
> Signed-off-by: Michael Niedermayer <michael at niedermayer.cc>
> ---
>  libavcodec/x86/cavsdsp.c | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
> 
> diff --git a/libavcodec/x86/cavsdsp.c b/libavcodec/x86/cavsdsp.c
> index 4b20e655a7..d5c63c5128 100644
> --- a/libavcodec/x86/cavsdsp.c
> +++ b/libavcodec/x86/cavsdsp.c
> @@ -565,7 +565,9 @@ av_cold void ff_cavsdsp_init_x86(CAVSDSPContext *c, AVCodecContext *avctx)
>  {
>      av_unused int cpu_flags = av_get_cpu_flags();
>  
> -    cavsdsp_init_mmx(c, avctx);
> +    if (INLINE_MMXEXT(cpu_flags))

This should be MMX, not MMXEXT. Also some of the functions are yasm/nasm, not inline.
Either add INLINE_MMX() and EXTERNAL_MMX() inside cavsdsp_init_mmx() where needed,
or use X86_MMX() here.

> +        cavsdsp_init_mmx(c, avctx);
> +
>  #if HAVE_AMD3DNOW_INLINE
>      if (INLINE_AMD3DNOW(cpu_flags))
>          cavsdsp_init_3dnow(c, avctx);
> 



More information about the ffmpeg-devel mailing list