-
Enhancement
-
Resolution: Won't Fix
-
P4
-
None
-
8-shenandoah, 11-shenandoah, 14, 15
Static analysis reveals "phase_process" local variable is unused here:
void ShenandoahTraversalGC::weak_refs_work_doit() {
...
ShenandoahPhaseTimings::Phase phase_process = ShenandoahPhaseTimings::weakrefs_process;
Indeed, we are missing ShenandoahGCPhase scope, like ShenandoahConcurrentMark::weak_refs_work_doit() has. New Traversal-specific counter needs to be added, and used with ShenandoahGCPhase here.
void ShenandoahTraversalGC::weak_refs_work_doit() {
...
ShenandoahPhaseTimings::Phase phase_process = ShenandoahPhaseTimings::weakrefs_process;
Indeed, we are missing ShenandoahGCPhase scope, like ShenandoahConcurrentMark::weak_refs_work_doit() has. New Traversal-specific counter needs to be added, and used with ShenandoahGCPhase here.
- relates to
-
JDK-8242082 Shenandoah: Purge Traversal mode
- Resolved