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