Skip to content

Commit ff61fb7

Browse files
committed
cache VerifyKillCFGUnusedNodes in graph
1 parent 2385a96 commit ff61fb7

File tree

2 files changed

+4
-2
lines changed

2 files changed

+4
-2
lines changed

compiler/src/jdk.graal.compiler/src/jdk/graal/compiler/graph/Graph.java

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -49,6 +49,7 @@
4949
import jdk.graal.compiler.graph.Node.ValueNumberable;
5050
import jdk.graal.compiler.graph.iterators.NodeIterable;
5151
import jdk.graal.compiler.graph.iterators.NodePredicate;
52+
import jdk.graal.compiler.nodes.util.GraphUtil;
5253
import jdk.graal.compiler.options.Option;
5354
import jdk.graal.compiler.options.OptionKey;
5455
import jdk.graal.compiler.options.OptionType;
@@ -83,6 +84,7 @@ private enum FreezeState {
8384
*/
8485
public final boolean verifyGraphs;
8586
public final boolean verifyGraphEdges;
87+
public final boolean verifyKillCFGUnusedNodes;
8688

8789
/**
8890
* Cached actual value of {@link GraalOptions#TrackNodeInsertion} to avoid expensive map lookup
@@ -335,6 +337,7 @@ public Graph(String name, OptionValues options, DebugContext debug, boolean trac
335337

336338
verifyGraphs = Options.VerifyGraalGraphs.getValue(options);
337339
verifyGraphEdges = Options.VerifyGraalGraphEdges.getValue(options);
340+
verifyKillCFGUnusedNodes = GraphUtil.Options.VerifyKillCFGUnusedNodes.getValue(options);
338341

339342
trackNodeInsertion = TrackNodeInsertion.getValue(options);
340343
}

compiler/src/jdk.graal.compiler/src/jdk/graal/compiler/nodes/util/GraphUtil.java

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -281,9 +281,8 @@ public static void killCFG(FixedNode node) {
281281
EconomicSet<Node> unusedNodes = null;
282282
EconomicSet<Node> unsafeNodes = null;
283283
Graph.NodeEventScope nodeEventScope = null;
284-
OptionValues options = node.getOptions();
285284
boolean verifyGraalGraphEdges = node.graph().verifyGraphEdges;
286-
boolean verifyKillCFGUnusedNodes = GraphUtil.Options.VerifyKillCFGUnusedNodes.getValue(options);
285+
boolean verifyKillCFGUnusedNodes = node.graph().verifyKillCFGUnusedNodes;
287286
if (verifyGraalGraphEdges) {
288287
unsafeNodes = collectUnsafeNodes(node.graph());
289288
}

0 commit comments

Comments
 (0)