On 2014-02-08 14:57:30 +0100, Diego Biurrun wrote: > On Sat, Feb 08, 2014 at 11:52:19AM +0100, Janne Grunau wrote: > > --- a/libavcodec/x86/dca.h > > +++ b/libavcodec/x86/dca.h > > @@ -18,7 +18,9 @@ > > > > -#if ARCH_X86_64 > > +#include "config.h" > > + > > +#if ARCH_X86_64 && HAVE_SSE2_INLINE > > # include "libavutil/x86/asm.h" > > # include "libavutil/mem.h" > > Why is this ARCH_X86_64 anyway?
Because only ARCH_X86_64 guarantees that SSE2 is available at runtime. We want also the specific HAVE_SSE2_INLINE since that's the only and correct way to disable this from configure with --disable-sse2 Janne _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
