[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