Request for reviews (XS): 6778657 and 6778662: Patchs from Red Hat

Vladimir Kozlov Vladimir.Kozlov at Sun.COM
Thu Dec 11 14:40:54 PST 2008

Web search shows that currently only ALPHA and IA64 use
/lib for 64-bits libraries. So the check will be:

#if defined(_LP64) && !defined(ALPHA) && !defined(IA64)


Tom Rodriguez wrote:
> That makes sense, though searching those locations when they don't exist 
> doesn't seem that bad.  Maybe specifying the exceptions instead of 
> enumerating the platforms where it applies would require fewer changes 
> going forward.  It's also more benign when it's wrong.
> #if defined(_LP64) && !defined(ALPHA)
> tom
> On Dec 11, 2008, at 10:09 AM, Christian Thalinger wrote:
>> On Thu, 2008-12-11 at 09:48 -0800, Tom Rodriguez wrote:
>>> Why isn't defined(_LP64) sufficient?  I don't know about S390 but it
>>> should be for AMD64 and 64 bit sparc.
>> I think this is because of multi-/biarch architectures.  Architectures
>> that only support 64-bit, like Alpha, don't have /lib64 or /usr/lib64.
>> Their libraries are simply in /lib and /usr/lib.
>> - Christian
jdk7 changes suggestion:

More information about the hotspot-compiler-dev mailing list