[MPlayer-dev-eng] Re: [PATCH] Various improvement in TOOLS/cpuinfo.c

[Guillaume POIRIER]

Hi,

On 5/15/06, Zuxy Meng <zuxy.meng at gmail.com> wrote:
> Hi,
>
> 2006/5/16, Guillaume POIRIER <poirierg at gmail.com>:
> > Hi,
> >
> >
> > Ok, good to know. The previous version (before last night) wasn't
> > working on amd64 anyhow, so at least your patch fixes a known
> > regression.
> >
> > I'm sure there's a way to support "macintel", amd64 and ia34...
>
> Will gcc define a macro like TARGET_64BIT under amd64?

Maybe it does, I have not been able to find much information about it
besides http://gcc.gnu.org/onlinedocs/gccint/Defining-Mode-Macros.html
and http://gcc.gnu.org/onlinedocs/gccint/Examples.html

The simplest way is probably to test on an amd64 machine.

In any case, no need to worry, the patch you sent today works on AMD64
(I just checked now as I'm home).


> > Also, do you know if the "b" asm contrain is supposed to work with gcc 2.95?
> >
>
> It should be. I checked linux kernel 2.4.9, which builds on gcc 2.95,
> and it contains the '=b' constraint. So I guess it's a problem
> specific to darwin gcc. Apple modifies gcc/config/i386/i386.h a lot.
>
> BTW: Is gcc 2.95 the earliest version we provide support?

Yes.

Guillaume

-- 
I am disillusioned enough to know that no man's opinion on any subject
is worth a damn unless backed up with enough genuine information to
make him really know what he's talking about.

-- H. P. Lovecraft (about the flamewars on FFmpeg and MPlayer-dev mailing lists)
http://www.brainyquote.com/quotes/quotes/h/hplovecr278144.html