Skip to content

Commit 0fec7ef

Browse files
committed
pass virtual frame to GetNameFromLocalNode
1 parent e800bce commit 0fec7ef

File tree

2 files changed

+5
-4
lines changed

2 files changed

+5
-4
lines changed

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/nodes/bytecode/GetNameFromLocalsNode.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -58,14 +58,15 @@
5858
import com.oracle.truffle.api.dsl.ImportStatic;
5959
import com.oracle.truffle.api.dsl.Specialization;
6060
import com.oracle.truffle.api.frame.Frame;
61+
import com.oracle.truffle.api.frame.VirtualFrame;
6162

6263
@GenerateUncached
6364
@ImportStatic(SpecialMethodSlot.class)
6465
public abstract class GetNameFromLocalsNode extends PNodeWithContext {
6566
public abstract Object execute(Frame frame, Object locals, String name, boolean cellvar);
6667

6768
@Specialization
68-
Object getValue(Frame frame, Object locals, String name, boolean isCellVar,
69+
Object getValue(VirtualFrame frame, Object locals, String name, boolean isCellVar,
6970
@Cached PyDictCheckExactNode checkDictNode,
7071
@Cached PyDictGetItem getDictItemNode,
7172
@Cached PyObjectGetItem getItemNode,

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/nodes/bytecode/PBytecodeRootNode.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -936,7 +936,7 @@ Object executeFromBci(VirtualFrame virtualFrame, Frame localFrame, Frame stackFr
936936
}
937937
case OpCodesConstants.LOAD_CLASSDEREF: {
938938
oparg |= Byte.toUnsignedInt(localBC[++bci]);
939-
stackTop = bytecodeLoadClassDefRef(localFrame, stackFrame, locals, stackTop, beginBci, localNodes, oparg);
939+
stackTop = bytecodeLoadClassDeref(virtualFrame, localFrame, stackFrame, locals, stackTop, beginBci, localNodes, oparg);
940940
break;
941941
}
942942
case OpCodesConstants.LOAD_DEREF: {
@@ -1539,7 +1539,7 @@ private int bytecodeStoreDeref(Frame localFrame, Frame stackFrame, int stackTop,
15391539
return stackTop;
15401540
}
15411541

1542-
private int bytecodeLoadClassDefRef(Frame localFrame, Frame stackFrame, Object locals, int stackTop, int bci, Node[] localNodes, int oparg) {
1542+
private int bytecodeLoadClassDeref(VirtualFrame virtualFrame, Frame localFrame, Frame stackFrame, Object locals, int stackTop, int bci, Node[] localNodes, int oparg) {
15431543
String name;
15441544
boolean isCellVar;
15451545
if (oparg < cellvars.length) {
@@ -1550,7 +1550,7 @@ private int bytecodeLoadClassDefRef(Frame localFrame, Frame stackFrame, Object l
15501550
isCellVar = false;
15511551
}
15521552
GetNameFromLocalsNode getNameFromLocals = insertChildNode(localNodes, bci, NODE_GET_NAME_FROM_LOCALS);
1553-
Object value = getNameFromLocals.execute(stackFrame, locals, name, isCellVar);
1553+
Object value = getNameFromLocals.execute(virtualFrame, locals, name, isCellVar);
15541554
if (value != null) {
15551555
stackFrame.setObject(++stackTop, value);
15561556
return stackTop;

0 commit comments

Comments
 (0)