On Thu, Oct 04, 2012 at 03:30:17PM +0100, Måns Rullgård wrote: > Diego Biurrun <di...@biurrun.de> writes: > > On Thu, Oct 04, 2012 at 02:04:27PM +0100, Måns Rullgård wrote: > >> Diego Biurrun <di...@biurrun.de> writes: > >> > --- a/libavutil/x86/cpu.c > >> > +++ b/libavutil/x86/cpu.c > >> > @@ -96,15 +113,7 @@ int ff_get_cpu_flags_x86(void) > >> > > >> > #if ARCH_X86_32 > >> > - x86_reg a, c; > >> > - > >> > - /* Check if CPUID is supported by attempting to toggle the ID bit in > >> > - * the EFLAGS register. */ > >> > - get_eflags(a); > >> > - set_eflags(a ^ 0x200000); > >> > - get_eflags(c); > >> > - > >> > - if (a == c) > >> > + if (!have_cpuid()) > >> > return 0; /* CPUID not supported */ > >> > #endif > >> > >> You could "#define have_cpuid() 1" for x86_64 and get rid of this #if. > > > > But I rely on have_cpuid to mean that I have either yasm or inline asm > > available in 5/5. > > I thought you tested for "cpuid" there.
No. Diego _______________________________________________ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel