[GR-40238] Skip instrumentation unit tests under bytecode interpreter

msimacek · msimacek · commit b09cd986b946 · 2022-08-08T11:04:34.000Z
PullRequest: graalpython/2383
diff --git a/ci.jsonnet b/ci.jsonnet
@@ -1 +1 @@
-{ "overlay": "9b526d09349134a4ed931935bbe2befedef81215" }
+{ "overlay": "17edbb52412c55f5a3903eededc75bf33371bd49" }
diff --git a/graalpython/com.oracle.graal.python.test/src/com/oracle/graal/python/test/PythonTests.java b/graalpython/com.oracle.graal.python.test/src/com/oracle/graal/python/test/PythonTests.java
@@ -48,15 +48,16 @@
 import java.util.Collections;
 import java.util.Map;
 
-import com.oracle.truffle.api.strings.TruffleString;
 import org.graalvm.polyglot.Context;
 import org.graalvm.polyglot.Engine;
 import org.graalvm.polyglot.PolyglotException;
 import org.graalvm.polyglot.Value;
 import org.junit.Assert;
+import org.junit.Assume;
 
 import com.oracle.graal.python.PythonLanguage;
 import com.oracle.graal.python.runtime.PythonContext;
+import com.oracle.graal.python.runtime.PythonOptions;
 import com.oracle.graal.python.runtime.PythonParser.ParserMode;
 import com.oracle.graal.python.runtime.exception.PException;
 import com.oracle.graal.python.test.interop.JavaInteropTest;
@@ -68,6 +69,7 @@
 import com.oracle.truffle.api.nodes.RootNode;
 import com.oracle.truffle.api.source.Source;
 import com.oracle.truffle.api.source.Source.LiteralBuilder;
+import com.oracle.truffle.api.strings.TruffleString;
 
 public class PythonTests {
     static {
@@ -132,6 +134,10 @@ public static void closeContext() {
         }
     }
 
+    public static void skipOnBytecodeInterpreter() {
+        Assume.assumeFalse(PythonOptions.EnableBytecodeInterpreter.getDefaultValue());
+    }
+
     public static void assertBenchNoError(Path scriptName, String[] args) {
         final ByteArrayOutputStream byteArrayErr = new ByteArrayOutputStream();
         final ByteArrayOutputStream byteArrayOut = new ByteArrayOutputStream();
diff --git a/graalpython/com.oracle.graal.python.test/src/com/oracle/graal/python/test/debug/PythonDebugTest.java b/graalpython/com.oracle.graal.python.test/src/com/oracle/graal/python/test/debug/PythonDebugTest.java
@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2018, 2021, Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 2018, 2022, Oracle and/or its affiliates. All rights reserved.
  * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
  *
  * The Universal Permissive License (UPL), Version 1.0
@@ -83,6 +83,7 @@ public void before() {
         newBuilder.allowAllAccess(true);
         PythonTests.closeContext();
         tester = new DebuggerTester(newBuilder);
+        PythonTests.skipOnBytecodeInterpreter();
     }
 
     @After
diff --git a/graalpython/com.oracle.graal.python.test/src/com/oracle/graal/python/test/interop/ReadAndWriteVariableTagsTests.java b/graalpython/com.oracle.graal.python.test/src/com/oracle/graal/python/test/interop/ReadAndWriteVariableTagsTests.java
@@ -50,6 +50,7 @@
 import org.graalvm.polyglot.Engine;
 import org.graalvm.polyglot.Instrument;
 import org.graalvm.polyglot.Source;
+import org.junit.Before;
 import org.junit.Test;
 
 import com.oracle.graal.python.nodes.PNode;
@@ -77,6 +78,11 @@ private static Context newContext(Engine engine) {
         return Context.newBuilder().allowExperimentalOptions(true).allowAllAccess(true).engine(engine).build();
     }
 
+    @Before
+    public void before() {
+        PythonTests.skipOnBytecodeInterpreter();
+    }
+
     @Test
     public void testInModule() throws Exception {
         String code = "a = 0\n" +
diff --git a/graalpython/com.oracle.graal.python.test/src/com/oracle/graal/python/test/interop/StandardTagsTests.java b/graalpython/com.oracle.graal.python.test/src/com/oracle/graal/python/test/interop/StandardTagsTests.java
@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2020, 2021, Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 2020, 2022, Oracle and/or its affiliates. All rights reserved.
  * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
  *
  * The Universal Permissive License (UPL), Version 1.0
@@ -45,6 +45,14 @@
 
 import java.util.HashMap;
 
+import org.graalvm.polyglot.Context;
+import org.graalvm.polyglot.Engine;
+import org.graalvm.polyglot.Instrument;
+import org.graalvm.polyglot.Source;
+import org.junit.Assert;
+import org.junit.Before;
+import org.junit.Test;
+
 import com.oracle.graal.python.nodes.PNode;
 import com.oracle.graal.python.nodes.function.InnerRootNode;
 import com.oracle.graal.python.test.PythonTests;
@@ -59,13 +67,6 @@
 import com.oracle.truffle.api.library.LibraryFactory;
 import com.oracle.truffle.api.nodes.Node;
 
-import org.graalvm.polyglot.Context;
-import org.graalvm.polyglot.Engine;
-import org.graalvm.polyglot.Instrument;
-import org.graalvm.polyglot.Source;
-import org.junit.Assert;
-import org.junit.Test;
-
 @SuppressWarnings("all")
 public class StandardTagsTests extends PythonTests {
 
@@ -75,6 +76,11 @@ private static Context newContext(Engine engine) {
         return Context.newBuilder().allowExperimentalOptions(true).allowAllAccess(true).engine(engine).build();
     }
 
+    @Before
+    public void before() {
+        PythonTests.skipOnBytecodeInterpreter();
+    }
+
     @Test
     public void testRootBodyTagModule() throws Exception {
         String code = "import time\n" +
diff --git a/graalpython/com.oracle.graal.python/src/com/oracle/graal/python/PythonLanguage.java b/graalpython/com.oracle.graal.python/src/com/oracle/graal/python/PythonLanguage.java
@@ -496,7 +496,7 @@ private CallTarget parseForBytecodeInterpreter(ParsingRequest request) {
             if (internal && !source.isInternal()) {
                 source = Source.newBuilder(source).internal(true).build();
             }
-            PBytecodeRootNode rootNode = new PBytecodeRootNode(this, code, source, null);
+            PBytecodeRootNode rootNode = PBytecodeRootNode.create(this, code, source);
             return PythonUtils.getOrCreateCallTarget(rootNode);
         }
         for (int optimize = 0; optimize < MIME_TYPE_EVAL.length; optimize++) {
@@ -533,7 +533,7 @@ public RootCallTarget parseForBytecodeInterpreter(PythonContext context, Source
             }
             CompilationUnit cu = compiler.compile(mod, EnumSet.noneOf(Compiler.Flags.class), optimize);
             CodeUnit co = cu.assemble();
-            RootNode rootNode = new PBytecodeRootNode(this, co, source, errorCb);
+            RootNode rootNode = PBytecodeRootNode.create(this, co, source, errorCb);
             GilNode gil = GilNode.getUncached();
             boolean wasAcquired = gil.acquire(context, rootNode);
             if (topLevel) {
diff --git a/graalpython/com.oracle.graal.python/src/com/oracle/graal/python/builtins/objects/code/CodeNodes.java b/graalpython/com.oracle.graal.python/src/com/oracle/graal/python/builtins/objects/code/CodeNodes.java
@@ -152,7 +152,7 @@ private RootCallTarget deserializeForBytecodeInterpreter(PythonLanguage language
                                 code.outputCanQuicken, code.variableShouldUnbox,
                                 code.generalizeInputsMap, code.generalizeVarsMap);
             }
-            RootNode rootNode = new PBytecodeRootNode(language, code, null, null);
+            RootNode rootNode = PBytecodeRootNode.create(language, code, null);
             if (code.isGeneratorOrCoroutine()) {
                 rootNode = new PBytecodeGeneratorFunctionRootNode(language, rootNode.getFrameDescriptor(), (PBytecodeRootNode) rootNode, code.name);
             }
diff --git a/graalpython/com.oracle.graal.python/src/com/oracle/graal/python/builtins/objects/code/PCode.java b/graalpython/com.oracle.graal.python/src/com/oracle/graal/python/builtins/objects/code/PCode.java
@@ -614,7 +614,7 @@ private static Object convertConstantToPythonSpace(RootNode rootNode, Object o)
         PythonObjectFactory factory = PythonObjectFactory.getUncached();
         if (o instanceof CodeUnit) {
             CodeUnit code = ((CodeUnit) o);
-            PBytecodeRootNode bytecodeRootNode = new PBytecodeRootNode(PythonLanguage.get(rootNode), code, getSourceSection(rootNode).getSource(), null);
+            PBytecodeRootNode bytecodeRootNode = PBytecodeRootNode.create(PythonLanguage.get(rootNode), code, getSourceSection(rootNode).getSource());
             return factory.createCode(bytecodeRootNode.getCallTarget(), bytecodeRootNode.getSignature(), code);
         } else if (o instanceof BigInteger) {
             return factory.createInt((BigInteger) o);
diff --git a/graalpython/com.oracle.graal.python/src/com/oracle/graal/python/lib/PyObjectGetMethod.java b/graalpython/com.oracle.graal.python/src/com/oracle/graal/python/lib/PyObjectGetMethod.java
@@ -65,6 +65,7 @@
 import com.oracle.truffle.api.dsl.Bind;
 import com.oracle.truffle.api.dsl.Cached;
 import com.oracle.truffle.api.dsl.Cached.Shared;
+import com.oracle.truffle.api.dsl.Fallback;
 import com.oracle.truffle.api.dsl.GenerateUncached;
 import com.oracle.truffle.api.dsl.ImportStatic;
 import com.oracle.truffle.api.dsl.Specialization;
@@ -105,18 +106,8 @@ protected static boolean isObjectGetAttribute(Object lazyClass) {
         return getattributeSlot == BuiltinMethodDescriptors.OBJ_GET_ATTRIBUTE && getattrSlot == PNone.NO_VALUE;
     }
 
-    @Specialization(guards = {"!isForeignObjectNode.execute(receiver)", "!isObjectGetAttribute(lazyClass)"}, limit = "1")
-    static Object getGenericAttr(Frame frame, Object receiver, TruffleString name,
-                    @SuppressWarnings("unused") @Shared("isForeign") @Cached IsForeignObjectNode isForeignObjectNode,
-                    @SuppressWarnings("unused") @Shared("getClassNode") @Cached GetClassNode getClass,
-                    @SuppressWarnings("unused") @Bind("getClass.execute(receiver)") Object lazyClass,
-                    @Cached PyObjectGetAttr getAttr) {
-        return new BoundDescriptor(getAttr.execute(frame, receiver, name));
-    }
-
-    @Specialization(guards = {"!isForeignObjectNode.execute(receiver)", "isObjectGetAttribute(lazyClass)", "name == cachedName"}, limit = "1")
+    @Specialization(guards = {"isObjectGetAttribute(lazyClass)" /* Implies not foreign */, "name == cachedName"}, limit = "1")
     static Object getFixedAttr(VirtualFrame frame, Object receiver, @SuppressWarnings("unused") TruffleString name,
-                    @SuppressWarnings("unused") @Shared("isForeign") @Cached IsForeignObjectNode isForeignObjectNode,
                     @SuppressWarnings("unused") @Shared("getClassNode") @Cached GetClassNode getClass,
                     @Bind("getClass.execute(receiver)") Object lazyClass,
                     @SuppressWarnings("unused") @Cached("name") TruffleString cachedName,
@@ -173,9 +164,8 @@ static Object getFixedAttr(VirtualFrame frame, Object receiver, @SuppressWarning
     }
 
     // No explicit branch profiling when we're looking up multiple things
-    @Specialization(guards = {"!isForeignObjectNode.execute(receiver)", "isObjectGetAttribute(lazyClass)"}, replaces = "getFixedAttr", limit = "1")
+    @Specialization(guards = "isObjectGetAttribute(lazyClass)" /* Implies not foreign */, replaces = "getFixedAttr", limit = "1")
     static Object getDynamicAttr(Frame frame, Object receiver, TruffleString name,
-                    @SuppressWarnings("unused") @Shared("isForeign") @Cached IsForeignObjectNode isForeignObjectNode,
                     @SuppressWarnings("unused") @Shared("getClassNode") @Cached GetClassNode getClass,
                     @Bind("getClass.execute(receiver)") Object lazyClass,
                     @Cached LookupAttributeInMRONode.Dynamic lookupNode,
@@ -217,7 +207,7 @@ static Object getDynamicAttr(Frame frame, Object receiver, TruffleString name,
         throw raiseNode.raise(AttributeError, ErrorMessages.OBJ_P_HAS_NO_ATTR_S, receiver, name);
     }
 
-    @Specialization(guards = "isForeignObjectNode.execute(receiver)", limit = "3")
+    @Specialization(guards = "isForeignObjectNode.execute(receiver)", limit = "1")
     Object getForeignMethod(VirtualFrame frame, Object receiver, TruffleString name,
                     @SuppressWarnings("unused") @Cached IsForeignObjectNode isForeignObjectNode,
                     @Cached TruffleString.ToJavaStringNode toJavaString,
@@ -238,4 +228,10 @@ Object getForeignMethod(VirtualFrame frame, Object receiver, TruffleString name,
             return new BoundDescriptor(getAttr.execute(frame, receiver, name));
         }
     }
+
+    @Fallback
+    static Object getGenericAttr(Frame frame, Object receiver, TruffleString name,
+                    @Cached PyObjectGetAttr getAttr) {
+        return new BoundDescriptor(getAttr.execute(frame, receiver, name));
+    }
 }
diff --git a/graalpython/com.oracle.graal.python/src/com/oracle/graal/python/nodes/bytecode/MakeFunctionNode.java b/graalpython/com.oracle.graal.python/src/com/oracle/graal/python/nodes/bytecode/MakeFunctionNode.java
@@ -144,7 +144,7 @@ int makeFunction(VirtualFrame frame, Object globals, int initialStackTop, int fl
 
     public static MakeFunctionNode create(PythonLanguage language, CodeUnit code, Source source) {
         RootCallTarget callTarget;
-        PBytecodeRootNode bytecodeRootNode = new PBytecodeRootNode(language, code, source, null);
+        PBytecodeRootNode bytecodeRootNode = PBytecodeRootNode.create(language, code, source);
         if (code.isGeneratorOrCoroutine()) {
             // TODO what should the frameDescriptor be? does it matter?
             callTarget = new PBytecodeGeneratorFunctionRootNode(language, bytecodeRootNode.getFrameDescriptor(), bytecodeRootNode, code.name).getCallTarget();
diff --git a/graalpython/com.oracle.graal.python/src/com/oracle/graal/python/nodes/bytecode/PBytecodeRootNode.java b/graalpython/com.oracle.graal.python/src/com/oracle/graal/python/nodes/bytecode/PBytecodeRootNode.java
@@ -50,6 +50,7 @@
 
 import java.math.BigInteger;
 import java.util.Arrays;
+import java.util.concurrent.locks.Lock;
 
 import com.oracle.graal.python.PythonLanguage;
 import com.oracle.graal.python.builtins.PythonBuiltinClassType;
@@ -492,10 +493,10 @@ public final class PBytecodeRootNode extends PRootNode implements BytecodeOSRNod
     @CompilationFinal private boolean usingCachedNodes;
     @CompilationFinal(dimensions = 1) private int[] conditionProfiles;
 
-    private static FrameDescriptor makeFrameDescriptor(CodeUnit co) {
+    private static FrameDescriptor makeFrameDescriptor(CodeUnit co, FrameInfo info) {
         int capacity = co.varnames.length + co.cellvars.length + co.freevars.length + co.stacksize + 1;
         FrameDescriptor.Builder newBuilder = FrameDescriptor.newBuilder(capacity);
-        newBuilder.info(new FrameInfo());
+        newBuilder.info(info);
         // locals
         for (int i = 0; i < co.varnames.length; i++) {
             TruffleString varname = co.varnames[i];
@@ -543,14 +544,22 @@ private static Signature makeSignature(CodeUnit co) {
     }
 
     @TruffleBoundary
-    public PBytecodeRootNode(TruffleLanguage<?> language, CodeUnit co, Source source, RaisePythonExceptionErrorCallback parserErrorCallback) {
-        this(language, makeFrameDescriptor(co), makeSignature(co), co, source, parserErrorCallback);
+    public static PBytecodeRootNode create(TruffleLanguage<?> language, CodeUnit co, Source source) {
+        return create(language, co, source, null);
     }
 
     @TruffleBoundary
-    public PBytecodeRootNode(TruffleLanguage<?> language, FrameDescriptor fd, Signature sign, CodeUnit co, Source source, RaisePythonExceptionErrorCallback parserErrorCallback) {
+    public static PBytecodeRootNode create(TruffleLanguage<?> language, CodeUnit co, Source source, RaisePythonExceptionErrorCallback parserErrorCallback) {
+        FrameInfo frameInfo = new FrameInfo();
+        FrameDescriptor fd = makeFrameDescriptor(co, frameInfo);
+        PBytecodeRootNode rootNode = new PBytecodeRootNode(language, fd, makeSignature(co), co, source, parserErrorCallback);
+        frameInfo.rootNode = rootNode;
+        return rootNode;
+    }
+
+    @TruffleBoundary
+    private PBytecodeRootNode(TruffleLanguage<?> language, FrameDescriptor fd, Signature sign, CodeUnit co, Source source, RaisePythonExceptionErrorCallback parserErrorCallback) {
         super(language, fd);
-        ((FrameInfo) fd.getInfo()).rootNode = this;
         this.celloffset = co.varnames.length;
         this.freeoffset = celloffset + co.cellvars.length;
         this.stackoffset = freeoffset + co.freevars.length;
@@ -659,9 +668,15 @@ private <A, T extends Node> T insertChildNode(Node[] nodes, int nodeIndex, Class
     @SuppressWarnings("unchecked")
     private <A, T extends Node> T doInsertChildNode(Node[] nodes, int nodeIndex, NodeFunction<A, T> nodeSupplier, A argument) {
         CompilerDirectives.transferToInterpreterAndInvalidate();
-        T newNode = nodeSupplier.apply(argument);
-        nodes[nodeIndex] = insert(newNode);
-        return newNode;
+        Lock lock = getLock();
+        lock.lock();
+        try {
+            T newNode = nodeSupplier.apply(argument);
+            nodes[nodeIndex] = insert(newNode);
+            return newNode;
+        } finally {
+            lock.unlock();
+        }
     }
 
     @SuppressWarnings("unchecked")
@@ -688,9 +703,15 @@ private <T extends Node, U> T insertChildNodeInt(Node[] nodes, int nodeIndex, Cl
     @SuppressWarnings("unchecked")
     private <T extends Node> T doInsertChildNodeInt(Node[] nodes, int nodeIndex, IntNodeFunction<T> nodeSupplier, int argument) {
         CompilerDirectives.transferToInterpreterAndInvalidate();
-        T newNode = nodeSupplier.apply(argument);
-        nodes[nodeIndex] = insert(newNode);
-        return newNode;
+        Lock lock = getLock();
+        lock.lock();
+        try {
+            T newNode = nodeSupplier.apply(argument);
+            nodes[nodeIndex] = insert(newNode);
+            return newNode;
+        } finally {
+            lock.unlock();
+        }
     }
 
     @SuppressWarnings("unchecked")
@@ -705,9 +726,15 @@ private <T extends Node, U extends T> U insertChildNode(Node[] nodes, int nodeIn
     @SuppressWarnings("unchecked")
     private <T extends Node> T doInsertChildNode(Node[] nodes, int nodeIndex, NodeSupplier<T> nodeSupplier) {
         CompilerDirectives.transferToInterpreterAndInvalidate();
-        T newNode = nodeSupplier.get();
-        nodes[nodeIndex] = insert(newNode);
-        return newNode;
+        Lock lock = getLock();
+        lock.lock();
+        try {
+            T newNode = nodeSupplier.get();
+            nodes[nodeIndex] = insert(newNode);
+            return newNode;
+        } finally {
+            lock.unlock();
+        }
     }
 
     @SuppressWarnings("unchecked")
diff --git a/graalpython/lib-graalpython/pip_hook.py b/graalpython/lib-graalpython/pip_hook.py
@@ -222,8 +222,11 @@ def apply_first_existing(dir, suffix, wd=None):
                 filename = first_existing(package_name, name_ver_match, dir, suffix)
                 if filename:
                     print("Patching package " + package_name + " using " + filename)
-                    patch_res = subprocess.run(["patch", "-f", "-d", location, "-p1", "-i", filename], cwd=wd)
-                    if patch_res.returncode != 0:
+                    try:
+                        subprocess.run(["patch", "-f", "-d", location, "-p1", "-i", filename], cwd=wd, check=True)
+                    except FileNotFoundError:
+                        print("WARNING: GraalPy needs the 'patch' utility to apply compatibility patches. Please install it using your system's package manager.")
+                    except subprocess.CalledProcessError:
                         print("Applying GraalPy patch failed for %s. The package may still work." % package_name)
                 elif os.path.isdir(dir):
                     patchfiles = [f for f in os.listdir(dir) if re.match("{0}{1}$".format(NAME_VER_PATTERN, suffix), f)]

Original file line number	Diff line number	Diff line change
`@@ -1 +1 @@`
`1`		`-{ "overlay": "9b526d09349134a4ed931935bbe2befedef81215" }`
	`1`	`+{ "overlay": "17edbb52412c55f5a3903eededc75bf33371bd49" }`
Original file line number	Diff line number	Diff line change
`@@ -1,5 +1,5 @@`
`1`	`1`	`/*`
`2`		`- * Copyright (c) 2018, 2021, Oracle and/or its affiliates. All rights reserved.`
	`2`	`+ * Copyright (c) 2018, 2022, Oracle and/or its affiliates. All rights reserved.`
`3`	`3`	`* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.`
`4`	`4`	`*`
`5`	`5`	`* The Universal Permissive License (UPL), Version 1.0`
`@@ -83,6 +83,7 @@ public void before() {`
`83`	`83`	`newBuilder.allowAllAccess(true);`
`84`	`84`	`PythonTests.closeContext();`
`85`	`85`	`tester = new DebuggerTester(newBuilder);`
	`86`	`+ PythonTests.skipOnBytecodeInterpreter();`
`86`	`87`	`}`
`87`	`88`
`88`	`89`	`@After`
Original file line number	Diff line number	Diff line change
`@@ -496,7 +496,7 @@ private CallTarget parseForBytecodeInterpreter(ParsingRequest request) {`
`496`	`496`	`if (internal && !source.isInternal()) {`
`497`	`497`	`source = Source.newBuilder(source).internal(true).build();`
`498`	`498`	`}`
`499`		`- PBytecodeRootNode rootNode = new PBytecodeRootNode(this, code, source, null);`
	`499`	`+ PBytecodeRootNode rootNode = PBytecodeRootNode.create(this, code, source);`
`500`	`500`	`return PythonUtils.getOrCreateCallTarget(rootNode);`
`501`	`501`	`}`
`502`	`502`	`for (int optimize = 0; optimize < MIME_TYPE_EVAL.length; optimize++) {`
`@@ -533,7 +533,7 @@ public RootCallTarget parseForBytecodeInterpreter(PythonContext context, Source`
`533`	`533`	`}`
`534`	`534`	`CompilationUnit cu = compiler.compile(mod, EnumSet.noneOf(Compiler.Flags.class), optimize);`
`535`	`535`	`CodeUnit co = cu.assemble();`
`536`		`- RootNode rootNode = new PBytecodeRootNode(this, co, source, errorCb);`
	`536`	`+ RootNode rootNode = PBytecodeRootNode.create(this, co, source, errorCb);`
`537`	`537`	`GilNode gil = GilNode.getUncached();`
`538`	`538`	`boolean wasAcquired = gil.acquire(context, rootNode);`
`539`	`539`	`if (topLevel) {`
Original file line number	Diff line number	Diff line change
`@@ -152,7 +152,7 @@ private RootCallTarget deserializeForBytecodeInterpreter(PythonLanguage language`
`152`	`152`	`code.outputCanQuicken, code.variableShouldUnbox,`
`153`	`153`	`code.generalizeInputsMap, code.generalizeVarsMap);`
`154`	`154`	`}`
`155`		`- RootNode rootNode = new PBytecodeRootNode(language, code, null, null);`
	`155`	`+ RootNode rootNode = PBytecodeRootNode.create(language, code, null);`
`156`	`156`	`if (code.isGeneratorOrCoroutine()) {`
`157`	`157`	`rootNode = new PBytecodeGeneratorFunctionRootNode(language, rootNode.getFrameDescriptor(), (PBytecodeRootNode) rootNode, code.name);`
`158`	`158`	`}`