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