RFR (M) 8200555: OopHandle should use Access API (original) (raw)

coleen.phillimore at oracle.com coleen.phillimore at oracle.com
Mon Apr 16 15:21:02 UTC 2018

On 4/16/18 10:26 AM, Erik Österlund wrote:

Hi Coleen,

This looks good to me. Thank you for accessorizing OopHandle. Thank you for the code review and discussions! Coleen

Thanks, /Erik On 2018-04-13 16:56, coleen.phillimore at oracle.com wrote: Summary: Add RootAccess<> to OopHandle.resolve() in runtime and interpreter code.  Add comments for compiler code for later.

For the current GCs this code does not generate additional code. Also, the compiler Access API code is TBD.   Since oopHandle.hpp erroneously included orderAccess.hpp and atomic.hpp, some include file fixing is also in this change. Tested with mach5 tier1,tier2 on all Oracle platforms.  Also tested compilation on linux-aarch64.  The other non-Oracle platforms weren't changed because they'll never need this. open webrev at http://cr.openjdk.java.net/~coleenp/8200555.01/webrev bug link https://bugs.openjdk.java.net/browse/JDK-8200555 Thanks, Coleen

More information about the hotspot-dev mailing list