Diego Biurrun <[email protected]> writes:

> @@ -60,18 +60,10 @@ av_cold void ff_yadif_init_x86(YADIFContext *yadif)
>  {
>      int cpu_flags = av_get_cpu_flags();
>  
> -#if HAVE_INLINE_ASM
> -#if HAVE_MMXEXT
> -    if (cpu_flags & AV_CPU_FLAG_MMXEXT)
> +    if (cpu_flags & AV_CPU_FLAG_MMXEXT && HAVE_MMXEXT_INLINE)
>          yadif->filter_line = yadif_filter_line_mmx2;
> -#endif
> -#if HAVE_SSE2
> -    if (cpu_flags & AV_CPU_FLAG_SSE2)
> +    if (cpu_flags & AV_CPU_FLAG_SSE2 && HAVE_SSE2_INLINE)
>          yadif->filter_line = yadif_filter_line_sse2;
> -#endif
> -#if HAVE_SSSE3
> -    if (cpu_flags & AV_CPU_FLAG_SSSE3)
> +    if (cpu_flags & AV_CPU_FLAG_SSSE3 && HAVE_SSSE3_INLINE)
>          yadif->filter_line = yadif_filter_line_ssse3;
> -#endif
> -#endif /* HAVE_INLINE_ASM */

The #ifdefs have to stay here.  If the features are not enabled, the
functions are not defined/declared at all, so referencing them even
under if(0) is an error.

-- 
Måns Rullgård
[email protected]
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to