changeset 19580:e7d46a5f177b

Add option DumpDuringGraphBuilding and default to false.
author Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
date Tue, 24 Feb 2015 13:23:52 +0100
parents 3df56ce39922
children 67d16e135ac2
files graal/com.oracle.graal.compiler.common/src/com/oracle/graal/compiler/common/GraalOptions.java graal/com.oracle.graal.java/src/com/oracle/graal/java/AbstractBytecodeParser.java graal/com.oracle.graal.java/src/com/oracle/graal/java/GraphBuilderPhase.java
diffstat 3 files changed, 10 insertions(+), 3 deletions(-) [+]
line wrap: on
line diff
--- a/graal/com.oracle.graal.compiler.common/src/com/oracle/graal/compiler/common/GraalOptions.java	Tue Feb 24 12:33:32 2015 +0100
+++ b/graal/com.oracle.graal.compiler.common/src/com/oracle/graal/compiler/common/GraalOptions.java	Tue Feb 24 13:23:52 2015 +0100
@@ -143,6 +143,9 @@
     @Option(help = "", type = OptionType.Debug)
     public static final OptionValue<String> PrintFilter = new OptionValue<>(null);
 
+    @Option(help = "", type = OptionType.Debug)
+    public static final StableOptionValue<Boolean> DumpDuringGraphBuilding = new StableOptionValue<>(false);
+
     // Debug settings:
     @Option(help = "", type = OptionType.Debug)
     public static final OptionValue<Boolean> BootstrapReplacements = new OptionValue<>(false);
--- a/graal/com.oracle.graal.java/src/com/oracle/graal/java/AbstractBytecodeParser.java	Tue Feb 24 12:33:32 2015 +0100
+++ b/graal/com.oracle.graal.java/src/com/oracle/graal/java/AbstractBytecodeParser.java	Tue Feb 24 13:23:52 2015 +0100
@@ -510,7 +510,6 @@
     private void genIfSame(Kind kind, Condition cond) {
         ValueNode y = frameState.pop(kind);
         ValueNode x = frameState.pop(kind);
-        // assert !x.isDeleted() && !y.isDeleted();
         genIf(x, cond, y);
     }
 
--- a/graal/com.oracle.graal.java/src/com/oracle/graal/java/GraphBuilderPhase.java	Tue Feb 24 12:33:32 2015 +0100
+++ b/graal/com.oracle.graal.java/src/com/oracle/graal/java/GraphBuilderPhase.java	Tue Feb 24 13:23:52 2015 +0100
@@ -308,7 +308,7 @@
                         index = iterateBlock(blocks, block);
                     }
 
-                    if (Debug.isDumpEnabled() && this.beforeReturnNode != startInstruction) {
+                    if (Debug.isDumpEnabled() && DumpDuringGraphBuilding.getValue() && this.beforeReturnNode != startInstruction) {
                         Debug.dump(currentGraph, "Bytecodes parsed: " + method.getDeclaringClass().getUnqualifiedName() + "." + method.getName());
                     }
                 }
@@ -333,7 +333,9 @@
                 context.peelIteration = this.getCurrentDimension();
                 context.targetPeelIteration = -1;
                 explodeLoopsContext.push(context);
-                Debug.dump(currentGraph, "before loop explosion " + context.peelIteration);
+                if (Debug.isDumpEnabled() && DumpDuringGraphBuilding.getValue()) {
+                    Debug.dump(currentGraph, "before loop explosion dimension " + context.peelIteration);
+                }
 
                 while (true) {
 
@@ -347,6 +349,9 @@
                         // We were reaching the backedge during explosion. Explode further.
                         context.peelIteration = context.targetPeelIteration;
                         context.targetPeelIteration = -1;
+                        if (Debug.isDumpEnabled() && DumpDuringGraphBuilding.getValue()) {
+                            Debug.dump(currentGraph, "next loop explosion iteration " + context.peelIteration);
+                        }
                     } else {
                         // We did not reach the backedge. Exit.
                         break;