Adam Jackson wrote:
> On Wed, 2009-12-16 at 16:45 -0800, Alan Coopersmith wrote:
>> Sun compilers don't include the trailing __ after __i386 and __amd64
>> like gcc, but do provide __x86 as a shorthand for __i386 || __amd64 .
> 
> Ennh.  It works, but it seems like we'd be better to do:
> 
> #if defined(__i386) && !defined(__i386__)
> #define __i386__ __i386
> #endif
> #if defined(__amd64) && !defined(__amd64__)
> #define __amd64__ __amd64
> #endif
> 
> in compiler.h in the sdk, and then use the __beforeandafter__ versions
> consistently.

I did that at first, but then hit other build failures in the PC98 mess.
That's probably fixable by ensuring compiler.h is consistently included
though.   I'll try poking at that some more.   (Do we still need PC98
support?)

-- 
        -Alan Coopersmith-           [email protected]
         Sun Microsystems, Inc. - X Window System Engineering

_______________________________________________
xorg-devel mailing list
[email protected]
http://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to