Fix for 8013633, fix platform ifdefs in unsafe.cpp
David Holmes
david.holmes at oracle.com
Mon May 6 20:46:48 PDT 2013
Hi Bill,
On 1/05/2013 4:55 AM, bill.pittore at oracle.com wrote:
> Rather than add yet another platform to the ifdefs in unsafe.cpp we
> thought it would be cleaner to reference
> #ifdef SUPPORTS_NATIVE_CX8
>
> which is set in platform specific globalsDefinitions_<arch>.hpp file
>
> Webrev is here:
> http://cr.openjdk.java.net/~bpittore/8013633/webrev.00/
Remove the comments
! // Sparc, X86, and ARM v7a have atomic jlong (8 bytes) instructions
Not much point moving away from a list of architectures in the ifdef if
you are going to add them to the comment anyway :)
Otherwise looks good to me.
Thanks,
David
> thanks,
> bill
More information about the hotspot-runtime-dev
mailing list