[MPlayer-dev-eng] [PATCH] fix explicitly setting CPU options in configure + cosmetics

[Alvaro Lopes]

Diego Biurrun wrote:

>Arpi told me on IRC he has mixed feelings about this patch - there has
>to be somebody else who can comment on this patch..  Please somebody
>speak up!
>
>Diego
>
>Diego Biurrun writes:
> > Resending.  Last time around I only got a comment from Alex who
> > pointed out a bug in the cosmetics part.  It's fixed now.
> > 
> > Hello Team!
> > 
> > I noticed some time ago that configure ignores CPU option settings
> > from the command line, like --disable-mmx or --disable-mtrr.  The
> > problem is that CPU feature detection etc happens before the main
> > command line argument parsing. 
>
These are two separate things, IMO. One is compile-time CPU options 
(which then should propagate through code via preprocessing directives), 
and other is run-time CPU options.

Compile-time options perhaps only make sense if we're enabling/disabling 
asm routines and/or gcc achitecture dependant flags. If we can build 
binaries with MMX support that will work on 386, the better.

So, I see no utility for this, unless as stated.

-- 

Álvaro Lopes 
---------------------
A .sig is just a .sig