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