RFR (S) 8243573: Shenandoah: rename GCParPhases and related code
Roman Kennke
rkennke at redhat.com
Fri Apr 24 19:47:04 UTC 2020
Looks good.
Thanks,
Roman
> RFE:
> https://bugs.openjdk.java.net/browse/JDK-8243573
>
> ShenandoahPhaseTimings::GCParPhases is awkward: it is plural and it carries unnecessary "GC" prefix.
> Should be just ShenandoahPhaseTimings::ParPhase. This also matches the local variables "par_phase"
> to be used to store it, in contrast with just "phase" for ShenandoahPhaseTimings::Phase.
>
> Webrev:
> https://cr.openjdk.java.net/~shade/8243573/webrev.01/
>
> Testing: hotspot_gc_shenandoah
>
More information about the hotspot-gc-dev
mailing list