RFR 8249837: Avoid direct or implicit Thread::current() calls when we already have a current thread variable
coleen.phillimore at oracle.com
coleen.phillimore at oracle.com
Thu Jul 30 14:03:23 UTC 2020
Thanks for the review, David.
Coleen
On 7/30/20 2:24 AM, David Holmes wrote:
> Hi Coleen,
>
> Thanks for taking on this one. :)
>
> On 30/07/2020 1:54 am, coleen.phillimore at oracle.com wrote:
>> Summary: Add current thread and/or moved HandleMark to the scope that
>> it is needed
>>
>> This patch removes HandleMarks from the GC code. I added a
>> NoHandleMark to these places and found the locations in the
>> verification code that needs the HandleMarks, and moved them there.
>
> Sounds good.
>
> I looked at all the other changes and they all seemed fine too.
>
> Thanks,
> David
>
>> Tested with tier1-6 with no failures.
>>
>> open webrev at
>> http://cr.openjdk.java.net/~coleenp/2020/8249837.01/webrev
>> bug link https://bugs.openjdk.java.net/browse/JDK-8249837
>>
>> Thanks,
>> Coleen
More information about the hotspot-dev
mailing list