[MPlayer-dev-eng] Re: -mieee CFLAG missing on alpha (patch included)

[Thomas Cort]

> I use Alpha, and MP3s work fine on it.  It may have been fixed
> in CVS.  I don't remember when it got fixed, but I recall that I
> was involved in the patch.

What CPU do you have? Alpha CPUs >= ev6 don't need -mieee. Mine is
ev56. Use cat /proc/cpuinfo to find out.

> Anyways, -mieee should *NOT* be used for CFLAGS.  I believe that
> it produces slower code (anybody know for sure?), which is a bad
> idea for processor-intensive programs.

"causes less than a 5% performance hit."
http://skarpsey.dyndns.org/alpha-lfs/#mieee

> Options that I endorse:
>  - get the CVS version
>  - release pre8

Where do I get pre8 the download page only has pre7?
http://www.mplayerhq.hu/homepage/design7/dload.html

The CVS version still doesn't compile:
cc -c -I../libvo -I../../libvo -I/usr/X11R6/include -O4  -mcpu=ev56
-pipe -ffast-math -fomit-frame-pointer -D_REENTRANT
-D_LARGEFILE_SOURCE -D_FILE_OFFSET_BITS=64  -I.. -I../loader       -o
stream_dvd.o stream_dvd.c
stream_dvd.c: In function `dvd_parse_chapter_range':
stream_dvd.c:82: warning: passing arg 2 of `strtol' from incompatible
pointer type
stream_dvd.c: At top level:
stream_dvd.c:420: error: parse error before '*' token
stream_dvd.c: In function `dvdtimetomsec':
stream_dvd.c:423: error: `dt' undeclared (first use in this function)
stream_dvd.c:423: error: (Each undeclared identifier is reported only once
stream_dvd.c:423: error: for each function it appears in.)
make[1]: *** [stream_dvd.o] Error 1
make[1]: Leaving directory `/root/mplayer-playground/main/libmpdemux'
make: *** [libmpdemux/libmpdemux.a] Error 2

-Thomas Cort