Mercurial > hg > graal-jvmci-8
changeset 19387:fc390dcb0ba5
Enable bytecode instruction level tracing only when assertions are enabled.
author | Thomas Wuerthinger <thomas.wuerthinger@oracle.com> |
---|---|
date | Sun, 15 Feb 2015 20:46:06 +0100 |
parents | 664b396dbe64 |
children | aadf2fd0a4a2 |
files | 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 | 2 files changed, 6 insertions(+), 4 deletions(-) [+] |
line wrap: on
line diff
--- a/graal/com.oracle.graal.java/src/com/oracle/graal/java/AbstractBytecodeParser.java Sun Feb 15 20:45:45 2015 +0100 +++ b/graal/com.oracle.graal.java/src/com/oracle/graal/java/AbstractBytecodeParser.java Sun Feb 15 20:46:06 2015 +0100 @@ -1148,10 +1148,11 @@ return frameState; } - protected void traceInstruction(int bci, int opcode, boolean blockStart) { + protected boolean traceInstruction(int bci, int opcode, boolean blockStart) { if (Debug.isEnabled() && Options.TraceBytecodeParserLevel.getValue() >= TRACELEVEL_INSTRUCTIONS && Debug.isLogEnabled()) { traceInstructionHelper(bci, opcode, blockStart); } + return true; } private void traceInstructionHelper(int bci, int opcode, boolean blockStart) {
--- a/graal/com.oracle.graal.java/src/com/oracle/graal/java/GraphBuilderPhase.java Sun Feb 15 20:45:45 2015 +0100 +++ b/graal/com.oracle.graal.java/src/com/oracle/graal/java/GraphBuilderPhase.java Sun Feb 15 20:46:06 2015 +0100 @@ -1584,8 +1584,8 @@ // read the opcode int opcode = stream.currentBC(); - traceState(); - traceInstruction(bci, opcode, bci == block.startBci); + assert traceState(); + assert traceInstruction(bci, opcode, bci == block.startBci); if (currentDepth == 0 && bci == entryBCI) { if (block.getJsrScope() != JsrScope.EMPTY_SCOPE) { throw new BailoutException("OSR into a JSR scope is not supported"); @@ -1648,10 +1648,11 @@ } } - private void traceState() { + private boolean traceState() { if (Debug.isEnabled() && Options.TraceBytecodeParserLevel.getValue() >= TRACELEVEL_STATE && Debug.isLogEnabled()) { traceStateHelper(); } + return true; } private void traceStateHelper() {