[15] 8246612: Shenandoah: add timing tracking to ShenandoahStringDedupRoots

Zhengyu Gu zgu at redhat.com
Thu Jun 4 19:02:37 UTC 2020



On 6/4/20 2:20 PM, Aleksey Shipilev wrote:
> On 6/4/20 8:18 PM, Zhengyu Gu wrote:
>> Please review this small patch that adds worker timing for
>> ShenandoahStringDedupRoots.
>>
>>
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8246612
>> Webrev: http://cr.openjdk.java.net/~zgu/JDK-8246612/weberv.00/index.html
> 
> *) Please use the argument name here, "phase", to match other uses:
>    212   ShenandoahConcurrentStringDedupRoots(ShenandoahPhaseTimings::Phase);

Fixed and pushed.

Thanks,

-Zhengyu

> 
> Otherwise looks good!
> 



More information about the hotspot-gc-dev mailing list