RFR: 8067790: Better support for native implementations of Atomic::cmpxchg

Kim Barrett kim.barrett at oracle.com
Thu Jan 1 07:43:48 UTC 2015


On Dec 17, 2014, at 11:38 AM, Erik Österlund <erik.osterlund at lnu.se> wrote:
> 
> RFE: https://bugs.openjdk.java.net/browse/JDK-8067790
> Webrev: http://cr.openjdk.java.net/~jwilhelm/8067790/webrev.00/

A couple more comments:

------------------------------------------------------------------------------ 

Note that once the fix for 8067306 is done, the STATIC_ASSERT tests
can be moved to namespace/global scope and no longer need to be
wrapped up in a dummy "test" function.

------------------------------------------------------------------------------ 

src/cpu/x86/vm/stubGenerator_x86_64.cpp 

Fixing the names in the "Support for ..." comments is fine, but they
should really be fixed, not partially updated.  Specifically,
"atomic::" => "Atomic::" should be applied throughout.




More information about the hotspot-dev mailing list