[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 shenandoah-dev
mailing list