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

Vladimir Kozlov Vladimir.Kozlov at Sun.COM
Thu Dec 11 10:04:28 PST 2008


I will go with #ifdef _LP64

Vladimir

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.
> 
> 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