# HG changeset patch # User Erik Eckstein # Date 1387438845 -3600 # Node ID 424e2bfecb722a194e4cdbed48f684c3fd9ba6ea # Parent 5f54b8a68346f05afa0d262ef471a5724cf223ca fix compiletime-expensive debug log message in LinearScan diff -r 5f54b8a68346 -r 424e2bfecb72 graal/com.oracle.graal.compiler/src/com/oracle/graal/compiler/alloc/LinearScanWalker.java --- a/graal/com.oracle.graal.compiler/src/com/oracle/graal/compiler/alloc/LinearScanWalker.java Thu Dec 19 08:35:37 2013 +0100 +++ b/graal/com.oracle.graal.compiler/src/com/oracle/graal/compiler/alloc/LinearScanWalker.java Thu Dec 19 08:40:45 2013 +0100 @@ -858,17 +858,14 @@ Interval interval = currentInterval; boolean result = true; - Indent indent = Debug.logAndIndent("activating interval %s, splitParent: %s, insertMoveWhenActivated: %b", interval.logString(allocator), interval.splitParent().operandNumber, - interval.insertMoveWhenActivated()); + Indent indent = Debug.logAndIndent("activating interval %s, splitParent: %d, insertMoveWhenActivated: %b", interval, interval.splitParent().operandNumber); final Value operand = interval.operand; if (interval.location() != null && isStackSlot(interval.location())) { // activating an interval that has a stack slot assigned . split it at first use // position // used for method parameters - if (getTraceLevel() >= 4) { - TTY.println(" interval has spill slot assigned (method parameter) . split it before first use"); - } + indent.log("interval has spill slot assigned (method parameter) . split it before first use"); splitStackInterval(interval); result = false; @@ -902,9 +899,7 @@ assert interval.isSplitChild(); assert interval.currentSplitChild() != null; assert !interval.currentSplitChild().operand.equals(operand) : "cannot insert move between same interval"; - if (getTraceLevel() >= 4) { - TTY.println("Inserting move from interval %d to %d because insertMoveWhenActivated is set", interval.currentSplitChild().operandNumber, interval.operandNumber); - } + indent.log("Inserting move from interval %d to %d because insertMoveWhenActivated is set", interval.currentSplitChild().operandNumber, interval.operandNumber); insertMove(interval.from(), interval.currentSplitChild(), interval); }