Skip to content

Commit fb10459

Browse files
committed
Remove ForceInlineGeneratorCalls option
Splitting generator calls might be make sense, but this was splitting generator functions.
1 parent 3a537cd commit fb10459

File tree

2 files changed

+0
-7
lines changed

2 files changed

+0
-7
lines changed

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/nodes/call/CallDispatchers.java

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -97,14 +97,10 @@ public static DirectCallNode createDirectCallNodeFor(PBuiltinFunction callee) {
9797

9898
@NeverDefault
9999
public static DirectCallNode createDirectCallNodeFor(PFunction callee) {
100-
boolean isGenerator = isGeneratorFunction(callee.getCallTarget());
101100
DirectCallNode callNode = Truffle.getRuntime().createDirectCallNode(callee.getCallTarget());
102101
if (callee.forceSplitDirectCalls()) {
103102
callNode.cloneCallTarget();
104103
}
105-
if (isGenerator && PythonLanguage.get(null).getEngineOption(PythonOptions.ForceInlineGeneratorCalls)) {
106-
callNode.forceInlining();
107-
}
108104
return callNode;
109105
}
110106

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/runtime/PythonOptions.java

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -270,9 +270,6 @@ public static void checkBytecodeDSLEnv() {
270270
@EngineOption @Option(category = OptionCategory.EXPERT, usageSyntax = "<limit>", help = "") //
271271
public static final OptionKey<Integer> NodeRecursionLimit = new OptionKey<>(1);
272272

273-
@EngineOption @Option(category = OptionCategory.EXPERT, usageSyntax = "true|false", help = "") //
274-
public static final OptionKey<Boolean> ForceInlineGeneratorCalls = new OptionKey<>(false);
275-
276273
@Option(category = OptionCategory.EXPERT, usageSyntax = "true|false", help = "Force to automatically import site.py module.", stability = OptionStability.STABLE) //
277274
public static final OptionKey<Boolean> ForceImportSite = new OptionKey<>(false);
278275

0 commit comments

Comments
 (0)