[PATCH] test: only test for double alignment on 64 bit. (#36986)
Alan Coopersmith
alan.coopersmith at oracle.com
Thu May 19 12:14:24 PDT 2011
On 05/19/11 12:58 AM, Keith Packard wrote:
> On Thu, 19 May 2011 14:54:28 +1000, Peter Hutterer <peter.hutterer at who-t.net> wrote:
>
>> because I naïvely thought that we needed double alignment on 32 bit too and
>> the test was written that way. Oh well, how about this patch instead then.
>> it just disables the alignment check on 32 bit.
>
> Seems sensible to me.
>
>> +#ifdef _LP64
>
> Do you have any idea what the difference between _LP64, _XSERVER64 and
> __LP64__ are? And whether we should be using any of these here (or
> elsewhere in the server)?
_LP64 is commonly defined by the platforms/compilers.
_XSERVER64 was our own invention before that became a defacto standard,
there used to be a monster if (__IA64__) || (__sparcv9__) || ... (every
other 64-bit RISC CPU) to decide if it should be #defined.
Now we have in include/xorg-server.h:
#ifdef _LP64
#define _XSERVER64 1
#endif
--
-Alan Coopersmith- alan.coopersmith at oracle.com
Oracle Solaris Platform Engineering: X Window System
More information about the xorg-devel
mailing list