# HG changeset patch # User tschatzl # Date 1550577360 -3600 # Node ID d85b3066ad2205a65818e7b28bb05554e90aeca2 # Parent 784537ff9c4e3eae8a8b20ef24e1aa57c1273d73 8218672: AOT code root scanning shows in the wrong position in the logs Reviewed-by: lkorinth, kbarrett diff -r 784537ff9c4e -r d85b3066ad22 src/hotspot/share/gc/g1/g1GCPhaseTimes.cpp --- a/src/hotspot/share/gc/g1/g1GCPhaseTimes.cpp Tue Feb 19 10:04:53 2019 +0100 +++ b/src/hotspot/share/gc/g1/g1GCPhaseTimes.cpp Tue Feb 19 12:56:00 2019 +0100 @@ -60,6 +60,9 @@ _gc_par_phases[SystemDictionaryRoots] = new WorkerDataArray(max_gc_threads, "SystemDictionary Roots (ms):"); _gc_par_phases[CLDGRoots] = new WorkerDataArray(max_gc_threads, "CLDG Roots (ms):"); _gc_par_phases[JVMTIRoots] = new WorkerDataArray(max_gc_threads, "JVMTI Roots (ms):"); +#if INCLUDE_AOT + _gc_par_phases[AOTCodeRoots] = new WorkerDataArray(max_gc_threads, "AOT Root Scan (ms):"); +#endif _gc_par_phases[CMRefRoots] = new WorkerDataArray(max_gc_threads, "CM RefProcessor Roots (ms):"); _gc_par_phases[WaitForStrongCLD] = new WorkerDataArray(max_gc_threads, "Wait For Strong CLD (ms):"); _gc_par_phases[WeakCLDRoots] = new WorkerDataArray(max_gc_threads, "Weak CLD Roots (ms):"); @@ -74,9 +77,6 @@ _gc_par_phases[ScanRS] = new WorkerDataArray(max_gc_threads, "Scan RS (ms):"); _gc_par_phases[OptScanRS] = new WorkerDataArray(max_gc_threads, "Optional Scan RS (ms):"); _gc_par_phases[CodeRoots] = new WorkerDataArray(max_gc_threads, "Code Root Scanning (ms):"); -#if INCLUDE_AOT - _gc_par_phases[AOTCodeRoots] = new WorkerDataArray(max_gc_threads, "AOT Root Scanning (ms):"); -#endif _gc_par_phases[ObjCopy] = new WorkerDataArray(max_gc_threads, "Object Copy (ms):"); _gc_par_phases[OptObjCopy] = new WorkerDataArray(max_gc_threads, "Optional Object Copy (ms):"); _gc_par_phases[Termination] = new WorkerDataArray(max_gc_threads, "Termination (ms):"); @@ -389,9 +389,6 @@ } debug_phase(_gc_par_phases[ScanRS]); debug_phase(_gc_par_phases[CodeRoots]); -#if INCLUDE_AOT - debug_phase(_gc_par_phases[AOTCodeRoots]); -#endif debug_phase(_gc_par_phases[ObjCopy]); debug_phase(_gc_par_phases[Termination]); debug_phase(_gc_par_phases[Other]); @@ -503,6 +500,9 @@ "SystemDictionaryRoots", "CLDGRoots", "JVMTIRoots", +#if INCLUDE_AOT + "AOTCodeRoots", +#endif "CMRefRoots", "WaitForStrongCLD", "WeakCLDRoots", @@ -512,9 +512,6 @@ "ScanRS", "OptScanRS", "CodeRoots", -#if INCLUDE_AOT - "AOTCodeRoots", -#endif "ObjCopy", "OptObjCopy", "Termination", diff -r 784537ff9c4e -r d85b3066ad22 src/hotspot/share/gc/g1/g1GCPhaseTimes.hpp --- a/src/hotspot/share/gc/g1/g1GCPhaseTimes.hpp Tue Feb 19 10:04:53 2019 +0100 +++ b/src/hotspot/share/gc/g1/g1GCPhaseTimes.hpp Tue Feb 19 12:56:00 2019 +0100 @@ -55,6 +55,9 @@ SystemDictionaryRoots, CLDGRoots, JVMTIRoots, +#if INCLUDE_AOT + AOTCodeRoots, +#endif CMRefRoots, WaitForStrongCLD, WeakCLDRoots, @@ -64,9 +67,6 @@ ScanRS, OptScanRS, CodeRoots, -#if INCLUDE_AOT - AOTCodeRoots, -#endif ObjCopy, OptObjCopy, Termination,