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

Igor Veresov Igor.Veresov at Sun.COM
Thu Dec 11 14:55:23 PST 2008


That's not quite true. Some Linux distributions have /lib for 64-bit libs. For 
example on Gentoo(amd64) I have: lib32, lib64, and lib symlinked to lib64.

igor

On Friday 12 December 2008 01:40:54 Vladimir Kozlov wrote:
> 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)
>
> Vladimir
>
> 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:
>
> http://webrev.invokedynamic.info/kvn/6778662/





More information about the hotspot-compiler-dev mailing list