# HG changeset patch # User Josef Eisl # Date 1435765179 -7200 # Node ID ca49ca6825a812bdf743c044fdac8c1f8ae665f8 # Parent 148053aa82084342937db106baac1172959deebe Use NodeValueMap in DebugInfoBuilder. diff -r 148053aa8208 -r ca49ca6825a8 graal/com.oracle.graal.compiler/src/com/oracle/graal/compiler/gen/DebugInfoBuilder.java --- a/graal/com.oracle.graal.compiler/src/com/oracle/graal/compiler/gen/DebugInfoBuilder.java Wed Jul 01 17:27:51 2015 +0200 +++ b/graal/com.oracle.graal.compiler/src/com/oracle/graal/compiler/gen/DebugInfoBuilder.java Wed Jul 01 17:39:39 2015 +0200 @@ -23,29 +23,30 @@ package com.oracle.graal.compiler.gen; import java.util.*; -import java.util.Map.Entry; +import java.util.Map.*; + +import com.oracle.graal.graph.*; +import com.oracle.graal.lir.*; +import com.oracle.graal.nodes.*; +import com.oracle.graal.nodes.spi.*; +import com.oracle.graal.nodes.util.*; +import com.oracle.graal.nodes.virtual.*; +import com.oracle.graal.virtual.nodes.*; import jdk.internal.jvmci.code.*; import jdk.internal.jvmci.common.*; import jdk.internal.jvmci.debug.*; import jdk.internal.jvmci.meta.*; -import com.oracle.graal.graph.*; -import com.oracle.graal.lir.*; -import com.oracle.graal.nodes.*; -import com.oracle.graal.nodes.util.*; -import com.oracle.graal.nodes.virtual.*; -import com.oracle.graal.virtual.nodes.*; - /** * Builds {@link LIRFrameState}s from {@link FrameState}s. */ public class DebugInfoBuilder { - protected final NodeMap nodeOperands; + protected final NodeValueMap nodeValueMap; - public DebugInfoBuilder(NodeMap nodeOperands) { - this.nodeOperands = nodeOperands; + public DebugInfoBuilder(NodeValueMap nodeValueMap) { + this.nodeValueMap = nodeValueMap; } protected final Map virtualObjects = Node.newMap(); @@ -216,7 +217,7 @@ } else if (value != null) { STATE_VARIABLES.increment(); - Value operand = nodeOperands.get(value); + Value operand = nodeValueMap.operand(value); assert operand != null && (operand instanceof Variable || operand instanceof JavaConstant) : operand + " for " + value; return operand; diff -r 148053aa8208 -r ca49ca6825a8 graal/com.oracle.graal.compiler/src/com/oracle/graal/compiler/gen/NodeLIRBuilder.java --- a/graal/com.oracle.graal.compiler/src/com/oracle/graal/compiler/gen/NodeLIRBuilder.java Wed Jul 01 17:27:51 2015 +0200 +++ b/graal/com.oracle.graal.compiler/src/com/oracle/graal/compiler/gen/NodeLIRBuilder.java Wed Jul 01 17:39:39 2015 +0200 @@ -100,15 +100,15 @@ public NodeLIRBuilder(StructuredGraph graph, LIRGeneratorTool gen) { this.gen = gen; this.nodeOperands = graph.createNodeMap(); - this.debugInfoBuilder = createDebugInfoBuilder(graph, nodeOperands); + this.debugInfoBuilder = createDebugInfoBuilder(graph, this); if (MatchExpressions.getValue()) { matchRules = MatchRuleRegistry.lookup(getClass()); } } - @SuppressWarnings({"unused", "hiding"}) - protected DebugInfoBuilder createDebugInfoBuilder(StructuredGraph graph, NodeMap nodeOperands) { - return new DebugInfoBuilder(nodeOperands); + @SuppressWarnings({"unused"}) + protected DebugInfoBuilder createDebugInfoBuilder(StructuredGraph graph, NodeValueMap nodeValueMap) { + return new DebugInfoBuilder(nodeValueMap); } /** diff -r 148053aa8208 -r ca49ca6825a8 graal/com.oracle.graal.hotspot.amd64/src/com/oracle/graal/hotspot/amd64/AMD64HotSpotNodeLIRBuilder.java --- a/graal/com.oracle.graal.hotspot.amd64/src/com/oracle/graal/hotspot/amd64/AMD64HotSpotNodeLIRBuilder.java Wed Jul 01 17:27:51 2015 +0200 +++ b/graal/com.oracle.graal.hotspot.amd64/src/com/oracle/graal/hotspot/amd64/AMD64HotSpotNodeLIRBuilder.java Wed Jul 01 17:39:39 2015 +0200 @@ -25,23 +25,24 @@ import static com.oracle.graal.hotspot.HotSpotBackend.*; import static jdk.internal.jvmci.amd64.AMD64.*; import static jdk.internal.jvmci.code.ValueUtil.*; + +import com.oracle.graal.compiler.amd64.*; +import com.oracle.graal.compiler.gen.*; +import com.oracle.graal.hotspot.*; +import com.oracle.graal.hotspot.nodes.*; +import com.oracle.graal.lir.*; +import com.oracle.graal.lir.amd64.AMD64Move.*; +import com.oracle.graal.lir.gen.*; +import com.oracle.graal.nodes.*; +import com.oracle.graal.nodes.CallTargetNode.*; +import com.oracle.graal.nodes.spi.*; + import jdk.internal.jvmci.amd64.*; import jdk.internal.jvmci.code.*; import jdk.internal.jvmci.debug.*; import jdk.internal.jvmci.hotspot.*; import jdk.internal.jvmci.meta.*; -import com.oracle.graal.compiler.amd64.*; -import com.oracle.graal.compiler.gen.*; -import com.oracle.graal.graph.*; -import com.oracle.graal.hotspot.*; -import com.oracle.graal.hotspot.nodes.*; -import com.oracle.graal.lir.*; -import com.oracle.graal.lir.amd64.AMD64Move.CompareAndSwapOp; -import com.oracle.graal.lir.gen.*; -import com.oracle.graal.nodes.CallTargetNode.InvokeKind; -import com.oracle.graal.nodes.*; - /** * LIR generator specialized for AMD64 HotSpot. */ @@ -62,9 +63,9 @@ } @Override - protected DebugInfoBuilder createDebugInfoBuilder(StructuredGraph graph, NodeMap nodeOperands) { + protected DebugInfoBuilder createDebugInfoBuilder(StructuredGraph graph, NodeValueMap nodeValueMap) { HotSpotLockStack lockStack = new HotSpotLockStack(gen.getResult().getFrameMapBuilder(), LIRKind.value(Kind.Long)); - return new HotSpotDebugInfoBuilder(nodeOperands, lockStack); + return new HotSpotDebugInfoBuilder(nodeValueMap, lockStack); } @Override diff -r 148053aa8208 -r ca49ca6825a8 graal/com.oracle.graal.hotspot.sparc/src/com/oracle/graal/hotspot/sparc/SPARCHotSpotNodeLIRBuilder.java --- a/graal/com.oracle.graal.hotspot.sparc/src/com/oracle/graal/hotspot/sparc/SPARCHotSpotNodeLIRBuilder.java Wed Jul 01 17:27:51 2015 +0200 +++ b/graal/com.oracle.graal.hotspot.sparc/src/com/oracle/graal/hotspot/sparc/SPARCHotSpotNodeLIRBuilder.java Wed Jul 01 17:39:39 2015 +0200 @@ -24,22 +24,23 @@ import static com.oracle.graal.hotspot.HotSpotBackend.*; import static jdk.internal.jvmci.sparc.SPARC.*; + +import com.oracle.graal.compiler.gen.*; +import com.oracle.graal.compiler.sparc.*; +import com.oracle.graal.hotspot.*; +import com.oracle.graal.hotspot.nodes.*; +import com.oracle.graal.lir.*; +import com.oracle.graal.lir.gen.*; +import com.oracle.graal.lir.sparc.SPARCMove.*; +import com.oracle.graal.nodes.*; +import com.oracle.graal.nodes.CallTargetNode.*; +import com.oracle.graal.nodes.spi.*; + import jdk.internal.jvmci.code.*; import jdk.internal.jvmci.debug.*; import jdk.internal.jvmci.hotspot.*; import jdk.internal.jvmci.meta.*; -import com.oracle.graal.compiler.gen.*; -import com.oracle.graal.compiler.sparc.*; -import com.oracle.graal.graph.*; -import com.oracle.graal.hotspot.*; -import com.oracle.graal.hotspot.nodes.*; -import com.oracle.graal.lir.*; -import com.oracle.graal.lir.gen.*; -import com.oracle.graal.lir.sparc.SPARCMove.CompareAndSwapOp; -import com.oracle.graal.nodes.CallTargetNode.InvokeKind; -import com.oracle.graal.nodes.*; - public class SPARCHotSpotNodeLIRBuilder extends SPARCNodeLIRBuilder implements HotSpotNodeLIRBuilder { private HotSpotGraalRuntimeProvider runtime; @@ -53,9 +54,9 @@ } @Override - protected DebugInfoBuilder createDebugInfoBuilder(StructuredGraph graph, NodeMap nodeOperands) { + protected DebugInfoBuilder createDebugInfoBuilder(StructuredGraph graph, NodeValueMap nodeValueMap) { HotSpotLockStack lockStack = new HotSpotLockStack(gen.getResult().getFrameMapBuilder(), LIRKind.value(Kind.Long)); - return new HotSpotDebugInfoBuilder(nodeOperands, lockStack); + return new HotSpotDebugInfoBuilder(nodeValueMap, lockStack); } private SPARCHotSpotLIRGenerator getGen() { diff -r 148053aa8208 -r ca49ca6825a8 graal/com.oracle.graal.hotspot/src/com/oracle/graal/hotspot/HotSpotDebugInfoBuilder.java --- a/graal/com.oracle.graal.hotspot/src/com/oracle/graal/hotspot/HotSpotDebugInfoBuilder.java Wed Jul 01 17:27:51 2015 +0200 +++ b/graal/com.oracle.graal.hotspot/src/com/oracle/graal/hotspot/HotSpotDebugInfoBuilder.java Wed Jul 01 17:39:39 2015 +0200 @@ -22,14 +22,15 @@ */ package com.oracle.graal.hotspot; +import static jdk.internal.jvmci.code.BytecodeFrame.*; + +import com.oracle.graal.compiler.gen.*; +import com.oracle.graal.nodes.*; +import com.oracle.graal.nodes.spi.*; + import jdk.internal.jvmci.code.*; import jdk.internal.jvmci.common.*; import jdk.internal.jvmci.meta.*; -import static jdk.internal.jvmci.code.BytecodeFrame.*; - -import com.oracle.graal.compiler.gen.*; -import com.oracle.graal.graph.*; -import com.oracle.graal.nodes.*; /** * Extends {@link DebugInfoBuilder} to allocate the extra debug information required for locks. @@ -38,8 +39,8 @@ private final HotSpotLockStack lockStack; - public HotSpotDebugInfoBuilder(NodeMap nodeOperands, HotSpotLockStack lockStack) { - super(nodeOperands); + public HotSpotDebugInfoBuilder(NodeValueMap nodeValueMap, HotSpotLockStack lockStack) { + super(nodeValueMap); this.lockStack = lockStack; }