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

Tom Rodriguez Thomas.Rodriguez at Sun.COM
Thu Dec 11 09:48:56 PST 2008


Why isn't defined(_LP64) sufficient?  I don't know about S390 but it  
should be for AMD64 and 64 bit sparc.

tom

On Dec 9, 2008, at 8:51 AM, Christian Thalinger wrote:

> On Tue, 2008-12-09 at 08:43 -0800, Vladimir Kozlov wrote:
>> http://webrev.invokedynamic.info/kvn/6778662/index.html
>>
>> Fixed 6778662: fixes directory search paths.
>
> Probably I should talk to Gary about that one, but shouldn't SPARC  
> also
> be added here (and MIPS/MIPSEL for Zero)?
>
> + #if defined(AMD64) || defined(_LP64) && (defined(PPC) ||  
> defined(S390))
>
> - Christian
>




More information about the hotspot-compiler-dev mailing list