JDK-8241063 : Shenandoah: Traversal weak references misses GC phase scope
  • Type: Enhancement
  • Component: hotspot
  • Sub-Component: gc
  • Affected Version: 8-shenandoah,11-shenandoah,14,15
  • Priority: P4
  • Status: Closed
  • Resolution: Won't Fix
  • Submitted: 2020-03-16
  • Updated: 2020-04-14
  • Resolved: 2020-04-14
Related Reports
Relates :  
Description
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.
Comments
Not needed after Traversal removal (JDK-8242082).
14-04-2020