Cache compiler phase name.

Author: woess

compiler/src/jdk.graal.compiler/src/jdk/graal/compiler/lir/phases/LIRPhase.java

@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2015, 2022, Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 2015, 2025, Oracle and/or its affiliates. All rights reserved.
  * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
  *
  * This code is free software; you can redistribute it and/or modify it
@@ -52,15 +52,7 @@ public static class Options {
         // @formatter:on
     }
 
-    /**
-     * Records time spent within {@link #apply}.
-     */
-    private final TimerKey timer;
-
-    /**
-     * Records memory usage within {@link #apply}.
-     */
-    private final MemUseTrackerKey memUseTracker;
+    private final LIRPhaseStatistics statistics;
 
     public static final class LIRPhaseStatistics {
         /**
@@ -73,6 +65,13 @@ public static final class LIRPhaseStatistics {
          */
         public final MemUseTrackerKey memUseTracker;
 
+        /**
+         * Cached phase name.
+         *
+         * @see LIRPhase#getName()
+         */
+        CharSequence phaseName;
+
         public LIRPhaseStatistics(Class<?> clazz) {
             timer = DebugContext.timer("LIRPhaseTime_%s", clazz);
             memUseTracker = DebugContext.memUseTracker("LIRPhaseMemUse_%s", clazz);
@@ -101,9 +100,7 @@ private static boolean checkName(CharSequence name) {
     }
 
     public LIRPhase() {
-        LIRPhaseStatistics statistics = getLIRPhaseStatistics(getClass());
-        timer = statistics.timer;
-        memUseTracker = statistics.memUseTracker;
+        this.statistics = getLIRPhaseStatistics(getClass());
     }
 
     /**
@@ -129,8 +126,8 @@ public final void apply(TargetDescription target, LIRGenerationResult lirGenRes,
         CharSequence name = getName();
         try (DebugContext.Scope s = debug.scope(name, this)) {
             try (CompilerPhaseScope cps = debug.enterCompilerPhase(name, null);
-                            DebugCloseable a = timer.start(debug);
-                            DebugCloseable c = memUseTracker.start(debug);
+                            DebugCloseable a = statistics.timer.start(debug);
+                            DebugCloseable c = statistics.memUseTracker.start(debug);
                             DebugCloseable d = gcStatistics(debug)) {
                 run(target, lirGenRes, context);
                 if (dumpLIR && debug.areScopesEnabled()) {
@@ -173,8 +170,13 @@ protected CharSequence createName() {
     }
 
     public final CharSequence getName() {
-        CharSequence name = createName();
+        CharSequence name = statistics.phaseName;
+        if (name != null) {
+            return name;
+        }
+        name = createName();
         assert checkName(name);
+        statistics.phaseName = name;
         return name;
     }
 }

compiler/src/jdk.graal.compiler/src/jdk/graal/compiler/phases/BasePhase.java

@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2011, 2023, Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 2011, 2025, Oracle and/or its affiliates. All rights reserved.
  * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
  *
  * This code is free software; you can redistribute it and/or modify it
@@ -246,6 +246,13 @@ public static class BasePhaseStatistics {
          */
         private final MemUseTrackerKey memUseTracker;
 
+        /**
+         * Cached phase name.
+         *
+         * @see BasePhase#getName()
+         */
+        CharSequence phaseName;
+
         public BasePhaseStatistics(Class<?> clazz) {
             timer = DebugContext.timer("PhaseTime_%s", clazz).doc("Time spent in phase.");
             executionCount = DebugContext.counter("PhaseCount_%s", clazz).doc("Number of phase executions.");
@@ -604,10 +611,20 @@ public void changed(NodeEvent e, Node node) {
         }
     }
 
-    public CharSequence getName() {
+    private CharSequence createName() {
         return new ClassTypeSequence(this.getClass());
     }
 
+    public CharSequence getName() {
+        CharSequence name = statistics.phaseName;
+        if (name != null) {
+            return name;
+        }
+        name = createName();
+        statistics.phaseName = name;
+        return name;
+    }
+
     protected abstract void run(StructuredGraph graph, C context);
 
     @Override

compiler/src/jdk.graal.compiler/src/jdk/graal/compiler/phases/ClassTypeSequence.java

@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2011, 2018, Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 2011, 2025, Oracle and/or its affiliates. All rights reserved.
  * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
  *
  * This code is free software; you can redistribute it and/or modify it
@@ -30,12 +30,13 @@
 import jdk.vm.ci.meta.ResolvedJavaType;
 
 /**
- * A printable representation of the name of class that can serialized as a fully qualified type for
- * dumping. This is to support deobfuscation of dump output. The {@link #toString()} is the
+ * A printable representation of the name of class that can be serialized as a fully qualified type
+ * for dumping. This is to support deobfuscation of dump output. The {@link #toString()} is the
  * unqualified name of the Class.
  */
 public final class ClassTypeSequence implements JavaType, CharSequence {
     private final Class<?> clazz;
+    private String simpleName;
 
     public ClassTypeSequence(Class<?> clazz) {
         this.clazz = clazz;
@@ -56,8 +57,14 @@ public String toJavaName(boolean qualified) {
         if (qualified) {
             return clazz.getName();
         } else {
+            String name = this.simpleName;
+            if (name != null) {
+                return name;
+            }
             int lastDot = clazz.getName().lastIndexOf('.');
-            return clazz.getName().substring(lastDot + 1);
+            name = clazz.getName().substring(lastDot + 1);
+            this.simpleName = name;
+            return name;
         }
     }