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

Aleksey Shipilev shade at redhat.com
Thu Jun 4 18:20:43 UTC 2020


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);

Otherwise looks good!

-- 
Thanks,
-Aleksey




More information about the hotspot-gc-dev mailing list