[MPlayer-dev-eng] [SURVEY] change default CPU target when compiled with runtime CPU detection

[Guillaume POIRIER]

Hi,
I've committed a patch to configure script to support VIA/Cyrix C2
CPU. C2 is really an El Cheapo cpu as it doesn't feature any buzz
words like SSE or 3dnow! (shame!).
Anyway, in order to support this little baby, we'd need to set default
"mcpu" from i686 to i586.

Here's the corresponding code in configure script (note the i686 is
_*probably*_ the most common CPU):

  else # if test "$_runtime_cpudetection" = no
    # i686 is probably the most common CPU - optimize for it
    _mcpu="$cpuopt=i686"
    # at least i486 required, for bswap instruction
    _march="-march=i486"


Well, I don't know what you ppl think about this: on the one hand such
a change may slow down (some special instruction of i686 can't be
used) MPlayer for most ppl who don't have such a toy, on the other
hand, why would we care as any sane person doesn't compile MPlayer
with runtime CPU detection.... which is wrong!
By default, debian dpkg script compiles MPlayer with runtime CPU
detection, which is IMHO very bad!

So here are the solutions I propose:

1) optimize to i586 by default and change the debian/rules script so
that MPlayer is not build with runtime CPU detection by default
2) optimize to i586 by default do not change debian/rules
3) do not change anything: C2 are not too common, and fools who
compile with runtime cpu detection are the one to blame.

I vote for 1), but what do you think ppl?

Guillaume
--
Reading doesn't hurt, really!
  -- Dominik 'Rathann' Mierzejewski