RFR (L, but tedious) 8188220: Remove Atomic::*_ptr() uses and overloads from hotspot
coleen.phillimore at oracle.com
coleen.phillimore at oracle.com
Tue Oct 10 22:01:01 UTC 2017
Summary: With the new template functions these are unnecessary.
The changes are mostly s/_ptr// and removing the cast to return type.
There weren't many types that needed to be improved to match the
template version of the function. Some notes:
1. replaced CASPTR with Atomic::cmpxchg() in mutex.cpp, rearranging
arguments.
2. renamed Atomic::replace_if_null to Atomic::cmpxchg_if_null. I
disliked the first name because it's not explicit from the callers that
there's an underlying cas. If people want to fight, I'll remove the
function and use cmpxchg because there are only a couple places where
this is a little nicer.
3. Added Atomic::sub()
Tested with JPRT, mach5 tier1-5 on linux,windows and solaris.
open webrev at http://cr.openjdk.java.net/~coleenp/8188220.01/webrev
bug link https://bugs.openjdk.java.net/browse/JDK-8188220
Thanks,
Coleen
More information about the hotspot-dev
mailing list