OptimizationUtility: if no global self time is available all compilation

units are hot

Author: davleopo

compiler/src/jdk.graal.compiler/src/jdk/graal/compiler/nodes/StructuredGraph.java

@@ -1337,17 +1337,20 @@ public OptimizationLog getOptimizationLog() {
      */
     public interface GlobalProfileProvider {
 
-        GlobalProfileProvider DEFAULT = new GlobalProfileProvider() {
+        /**
+         * The default time returned when no global profile provider is available on the platform.
+         */
+        int GLOBAL_PROFILE_PROVIDER_DISABLED = -1;
 
-            public static final int DEFAULT_TIME = -1;
+        GlobalProfileProvider DEFAULT = new GlobalProfileProvider() {
 
             /**
              * The default time provider always returns -1, i.e. the self time is unknown by
              * default.
              */
             @Override
             public double getGlobalSelfTimePercent() {
-                return DEFAULT_TIME;
+                return GLOBAL_PROFILE_PROVIDER_DISABLED;
             }
 
             /**

compiler/src/jdk.graal.compiler/src/jdk/graal/compiler/phases/common/util/OptimizationUtility.java

@@ -49,11 +49,17 @@ public static <X> X chooseAdaptiveBudgetFactor(StructuredGraph graph, X coldValu
     }
 
     /**
-     * Determine if the given graph should be considered "hot" for additional optimization purposes.
-     * We define "hot" by inspecting its self time with respect to overall execution time. This is a
-     * pure heuristical value.
+     * Determine if the given graph should be considered "hot" for optimization purposes. We define
+     * "hot" by inspecting its self time with respect to overall execution time. This is a purely
+     * heuristical value.
      */
     public static boolean hotGlobalSelfTime(StructuredGraph graph) {
-        return graph.globalProfileProvider().getGlobalSelfTimePercent() > Options.HotCodeMinSelfTime.getValue(graph.getOptions());
+        final double globalSelfTimePercent = graph.globalProfileProvider().getGlobalSelfTimePercent();
+        if (globalSelfTimePercent == StructuredGraph.GlobalProfileProvider.GLOBAL_PROFILE_PROVIDER_DISABLED) {
+            // We are in a mode where there is no self time data available. In JIT all compilation
+            // units are hot.
+            return true;
+        }
+        return globalSelfTimePercent > Options.HotCodeMinSelfTime.getValue(graph.getOptions());
     }
 }