RFR (M) 8200555: OopHandle should use Access API

Erik Österlund erik.osterlund at oracle.com
Mon Apr 16 14:26:00 UTC 2018


Hi Coleen,

This looks good to me. Thank you for accessorizing OopHandle.

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