When a call target supplier is run, that needs to be behind a boundary

This only applies for some implementations of the supplier, so instead
of making this the job of the user of the supplier, or making it the
same for all suppliers, have the implementor do it.

Co-authored-by: Matthew Alt <[email protected]>
Co-authored-by: Kevin Menard <[email protected]>

Author: 3 people

src/main/java/org/truffleruby/builtins/CoreMethodNodeManager.java

@@ -262,7 +262,10 @@ private static void addMethod(
             } else {
                 if (context.getLanguageSlow().options.LAZY_CALLTARGETS) {
                     callTarget = null;
-                    callTargetSupplier = new CachedSupplier<>(() -> callTargetFactory.apply(sharedMethodInfo));
+                    callTargetSupplier = new CachedSupplier<>(() -> {
+                        CompilerDirectives.transferToInterpreterAndInvalidate();
+                        return callTargetFactory.apply(sharedMethodInfo);
+                    });
                 } else {
                     callTarget = callTargetFactory.apply(sharedMethodInfo);
                     callTargetSupplier = null;

src/main/java/org/truffleruby/language/methods/InternalMethod.java

@@ -11,7 +11,6 @@
 
 import java.util.Set;
 
-import com.oracle.truffle.api.CompilerDirectives;
 import com.oracle.truffle.api.CompilerDirectives.CompilationFinal;
 import com.oracle.truffle.api.dsl.NodeFactory;
 import org.truffleruby.RubyContext;
@@ -233,7 +232,6 @@ public RubyProc getProc() {
 
     public RootCallTarget getCallTarget() {
         if (callTarget == null) {
-            CompilerDirectives.transferToInterpreterAndInvalidate();
             callTarget = callTargetSupplier.get();
             assert RubyRootNode.of(callTarget).getSharedMethodInfo() == sharedMethodInfo;
         }

src/main/java/org/truffleruby/parser/MethodTranslator.java

@@ -12,6 +12,7 @@
 import java.util.Arrays;
 import java.util.function.Supplier;
 
+import com.oracle.truffle.api.CompilerDirectives;
 import org.truffleruby.RubyLanguage;
 import org.truffleruby.collections.CachedSupplier;
 import org.truffleruby.core.IsNilNode;
@@ -423,6 +424,7 @@ public CachedSupplier<RootCallTarget> buildMethodNodeCompiler(SourceIndexLength
 
         if (shouldLazyTranslate) {
             return new CachedSupplier<>(() -> {
+                CompilerDirectives.transferToInterpreterAndInvalidate();
                 return translateMethodNode(sourceSection, defNode, bodyNode).getCallTarget();
             });
         } else {