[MPlayer-dev-eng] [PATCH/RFC] libswscale using mmx even when disabled by configure
Diego Biurrun
diego at biurrun.de
Wed Jul 19 13:02:13 CEST 2006
On Wed, Jul 19, 2006 at 01:13:43AM -0400, Rich Felker wrote:
> while testing just now, i discovered that 3 files in libswscale try to
> assemble mmx code even when mmx is explicitly disabled. they're
> wrongly using ARCH_X86[_64] defines instead of HAVE_{MMX|MMX2|3DNOW}
> to decide what to compile. i was having trouble sorting through all
> the ifdef mess so i just undefined ARCH_X86 when mmx, mmx2, and 3dnow
> are all disabled. this is not really a clean fix though.
Hey, there are much worse files ;-p
Why not check for HAVE_MMX (or whatever it's called) instead? It's a
few more checks, but it's the only way to be correct.
> on the other hand, what about having a global --disable-asm or
> --disable-arch option in the configure script to disable the setting
> of the ARCH_* defines entirely?
If you find it useful, go ahead and implement it...
Diego
More information about the MPlayer-dev-eng
mailing list