Diego Biurrun <di...@biurrun.de> writes: > 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.
Why not? -- Måns Rullgård m...@mansr.com _______________________________________________ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel
