Fold one branch in RubyArguments.getDeclarationFrame() for host inlining

eregon · eregon · commit 17e85f642032 · 2023-07-27T12:12:38.000+02:00
* 4503 -&gt; 4412 for org.truffleruby.language.dispatch.RubyCallNode.execute
* 106 + 202 -&gt; 106 + 106:
  INLINE   org.truffleruby.language.arguments.RubyArguments.getDeclarationFrame(Frame, int)                cost  106
    INLINE   org.truffleruby.language.arguments.RubyArguments.getDeclarationFrame(MaterializedFrame, int)  cost  202
  After:
  INLINE   org.truffleruby.language.arguments.RubyArguments.getDeclarationFrame(Frame, int)                cost  106
    INLINE   org.truffleruby.language.arguments.RubyArguments.getDeclarationFrame(MaterializedFrame, int)  cost  106
diff --git a/src/main/java/org/truffleruby/language/arguments/RubyArguments.java b/src/main/java/org/truffleruby/language/arguments/RubyArguments.java
@@ -386,7 +386,7 @@ public static MaterializedFrame getDeclarationFrame(MaterializedFrame frame, int
         assert level >= 0;
 
         CompilerAsserts.partialEvaluationConstant(level);
-        return level <= RubyBaseNode.MAX_EXPLODE_SIZE
+        return (CompilerDirectives.inCompiledCode() && level <= RubyBaseNode.MAX_EXPLODE_SIZE)
                 ? getDeclarationFrameExplode(frame, level)
                 : getDeclarationFrameLoop(frame, level);
     }

Original file line number	Diff line number	Diff line change
`@@ -386,7 +386,7 @@ public static MaterializedFrame getDeclarationFrame(MaterializedFrame frame, int`
`386`	`386`	`assert level >= 0;`
`387`	`387`
`388`	`388`	`CompilerAsserts.partialEvaluationConstant(level);`
`389`		`- return level <= RubyBaseNode.MAX_EXPLODE_SIZE`
	`389`	`+ return (CompilerDirectives.inCompiledCode() && level <= RubyBaseNode.MAX_EXPLODE_SIZE)`
`390`	`390`	`? getDeclarationFrameExplode(frame, level)`
`391`	`391`	`: getDeclarationFrameLoop(frame, level);`
`392`	`392`	`}`