Add test

Author: tkrodriguez

compiler/src/jdk.graal.compiler.test/src/jdk/graal/compiler/debug/test/DebugContextTest.java

@@ -337,4 +337,27 @@ public void testDebugConfig() {
         Assert.assertTrue(options2.toString().contains("MethodFilter=test"));
         Assert.assertTrue(options2.toString().contains("DumpOnError=true"));
     }
+
+    @Test
+    public void testIsCountEnabled1() {
+        EconomicMap<OptionKey<?>, Object> map = EconomicMap.create();
+        map.put(DebugOptions.Count, "");
+        OptionValues options = new OptionValues(map);
+        DebugContext debug = new Builder(options).build();
+        try (Scope s1 = debug.scope("Scope")) {
+            Assert.assertTrue(debug.isCountEnabled());
+        }
+    }
+
+    @Test
+    public void testIsCountEnabled2() {
+        EconomicMap<OptionKey<?>, Object> map = EconomicMap.create();
+        map.put(DebugOptions.Counters, "");
+        OptionValues options = new OptionValues(map);
+        DebugContext debug = new Builder(options).build();
+        try (Scope s1 = debug.scope("Scope")) {
+            Assert.assertTrue(debug.isCountEnabled());
+        }
+    }
+
 }

compiler/src/jdk.graal.compiler/src/jdk/graal/compiler/debug/DebugConfigImpl.java

@@ -160,17 +160,15 @@ public PrintStream output() {
     }
 
     private boolean isEnabled(DebugContext.Scope scope, DebugFilter filter) {
-        return getLevel(scope, filter) >= 0;
+        return getLevel(scope, filter) >= 1;
     }
 
     private int getLevel(DebugContext.Scope scope, DebugFilter filter) {
-        int level;
         if (filter == null) {
             // null means the value has not been set
-            level = -1;
-        } else {
-            level = filter.matchLevel(scope);
+            return -1;
         }
+        int level = filter.matchLevel(scope);
         if (level >= 0 && !checkMethodFilter(scope)) {
             level = -1;
         }