RFR (M) 8200555: OopHandle should use Access API

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