Skip to content

Commit c9bffa0

Browse files
committed
Rename to rootMethodScope
1 parent c20fe94 commit c9bffa0

File tree

3 files changed

+8
-8
lines changed

3 files changed

+8
-8
lines changed

compiler/src/jdk.graal.compiler/src/jdk/graal/compiler/nodes/GraphDecoder.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2156,9 +2156,9 @@ protected Object readObject(MethodScope methodScope) {
21562156
/**
21572157
* Removes unnecessary nodes from the graph after decoding.
21582158
*
2159-
* @param methodScope The current method.
2159+
* @param rootMethodScope The current method.
21602160
*/
2161-
protected void cleanupGraph(MethodScope methodScope) {
2161+
protected void cleanupGraph(MethodScope rootMethodScope) {
21622162
for (MergeNode merge : graph.getNodes(MergeNode.TYPE)) {
21632163
for (ProxyPlaceholder placeholder : merge.usages().filter(ProxyPlaceholder.class).snapshot()) {
21642164
placeholder.replaceAndDelete(placeholder.value);

compiler/src/jdk.graal.compiler/src/jdk/graal/compiler/nodes/SimplifyingGraphDecoder.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -141,9 +141,9 @@ public SimplifyingGraphDecoder(Architecture architecture, StructuredGraph graph,
141141
}
142142

143143
@Override
144-
protected void cleanupGraph(MethodScope methodScope) {
144+
protected void cleanupGraph(MethodScope rootMethodScope) {
145145
GraphUtil.normalizeLoops(graph);
146-
super.cleanupGraph(methodScope);
146+
super.cleanupGraph(rootMethodScope);
147147

148148
/*
149149
* To avoid calling tryKillUnused for each individual floating node we remove during
@@ -152,7 +152,7 @@ protected void cleanupGraph(MethodScope methodScope) {
152152
*/
153153
final NodeBitMap unusedNodes = new NodeBitMap(graph);
154154

155-
for (Node node : graph.getNewNodes(methodScope.methodStartMark)) {
155+
for (Node node : graph.getNewNodes(rootMethodScope.methodStartMark)) {
156156
if (node instanceof MergeNode mergeNode) {
157157
if (mergeNode.forwardEndCount() == 1) {
158158
graph.reduceTrivialMerge(mergeNode, false, unusedNodes);

compiler/src/jdk.graal.compiler/src/jdk/graal/compiler/replacements/PEGraphDecoder.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -917,8 +917,8 @@ protected PEMethodScope createMethodScope(StructuredGraph targetGraph, PEMethodS
917917
}
918918

919919
@Override
920-
protected void cleanupGraph(MethodScope methodScope) {
921-
super.cleanupGraph(methodScope);
920+
protected void cleanupGraph(MethodScope rootMethodScope) {
921+
super.cleanupGraph(rootMethodScope);
922922

923923
for (FrameState frameState : graph.getNodes(FrameState.TYPE)) {
924924
if (frameState.bci == BytecodeFrame.UNWIND_BCI) {
@@ -928,7 +928,7 @@ protected void cleanupGraph(MethodScope methodScope) {
928928
* anything because the usages of the frameState are not available yet. So we need
929929
* to call it again.
930930
*/
931-
PEMethodScope peMethodScope = (PEMethodScope) methodScope;
931+
PEMethodScope peMethodScope = (PEMethodScope) rootMethodScope;
932932
Invoke invoke = peMethodScope.invokeData != null ? peMethodScope.invokeData.invoke : null;
933933
InliningUtil.handleMissingAfterExceptionFrameState(frameState, invoke, null, true);
934934

0 commit comments

Comments
 (0)