[MPlayer-dev-eng] [PATCH] Athlon 64 configure script support

Guillaume POIRIER poirierg at gmail.com
Sat May 28 17:27:50 CEST 2005


Hi,

On 5/27/05, Corey Hickey <bugfood-ml at fatooh.org> wrote:
> Guillaume POIRIER wrote:
> > Hi,
> >
> > On 5/27/05, Guillaume POIRIER <poirierg at gmail.com> wrote:
> > [..]
> >
> >>I plan to commit this patch friday around noon (GMT). If you see
> >>obsious problems with it, please speak up so we can discuss it.
> >>Otherwise, it could still be fixed later on.
> >
> >
> > Committed. Thanks
> >
> > Guillaume
> >
> 
> Actually, the patch I wrote was separate from Robert Swain's patch. His
> patch was for 32-bit, mine was for 64-bit.

right. I did not notice that. I did test both patch in 32-bit mode for
Rob's and 64-bit mode for yours, and they worked fine.


> I've attached a different patch for 32-bit. This one, rather than
> checking the compiler version, goes ahead and sets proc=athlon64; I
> added a test to the optimization check so that if proc=athlon64 doesn't
> work with the current compiler it'll fall back to athlon-xp. I think
> doing it this way a little cleaner, and definitely consistant with the
> way configure is written right now.
> 
> Please note that I HAVE NOT TESTED THIS PATCH because I don't have a
> fully-functional 32-bit chroot anymore. It really ought to work, though.

Works as expected. Committed.

Guillaume




More information about the MPlayer-dev-eng mailing list