Mercurial > hg > graal-compiler
changeset 13463:424e2bfecb72
fix compiletime-expensive debug log message in LinearScan
author | Erik Eckstein <erik.eckstein@oracle.com> |
---|---|
date | Thu, 19 Dec 2013 08:40:45 +0100 |
parents | 5f54b8a68346 |
children | 2236d18302e0 |
files | graal/com.oracle.graal.compiler/src/com/oracle/graal/compiler/alloc/LinearScanWalker.java |
diffstat | 1 files changed, 3 insertions(+), 8 deletions(-) [+] |
line wrap: on
line diff
--- 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); }