[GR-18163] Skip specs for inline cache when --engine.Splitting=false

PullRequest: truffleruby/4293

Author: andrykonchin

spec/truffle/regexp/inline_caching_spec.rb

@@ -13,7 +13,9 @@
 # This test requires splitting (--engine.Splitting) which is only available with the OptimizedTruffleRuntime.
 # It fails under --experimental-engine-caching because CallInternalMethodNode does not have cached specializations for
 # !isSingleContext() and so ends up using an IndirectCallNode which prevents splitting.
-guard -> { TruffleRuby.jit? && !Truffle::Boot.get_option('experimental-engine-caching') && Truffle::Boot.get_option("default-cache") != 0 } do
+guard -> { Primitive.vm_splitting_enabled? &&
+  !Truffle::Boot.get_option('experimental-engine-caching') &&
+  Truffle::Boot.get_option("default-cache") != 0 } do
   describe "Inline caching for dynamically-created Regexp works for" do
     before :each do
       @performance_warnings, Warning[:performance] = Warning[:performance], true

src/main/java/org/truffleruby/core/CoreLibrary.java

@@ -23,6 +23,7 @@
 import java.util.concurrent.ConcurrentMap;
 
 import com.oracle.truffle.api.RootCallTarget;
+import com.oracle.truffle.api.nodes.DirectCallNode;
 import org.graalvm.shadowed.org.jcodings.transcode.EConvFlags;
 import org.truffleruby.RubyContext;
 import org.truffleruby.RubyLanguage;
@@ -253,6 +254,8 @@ public final class CoreLibrary {
      * using require_relative and the patch location is not enough to find the correct file. */
     private final ConcurrentMap<String, String> originalRequires;
 
+    private DirectCallNode callNodeToCheckSplittingEnabled;
+
     @TruffleBoundary
     private static SourceSection initCoreSourceSection() {
         final Source.SourceBuilder builder = Source.newBuilder(TruffleRuby.LANGUAGE_ID, "", "(core)");
@@ -933,6 +936,15 @@ public boolean isTruffleBootMainMethod(SharedMethodInfo info) {
         return info == truffleBootMainInfo;
     }
 
+    public boolean isSplittingEnabled() {
+        if (callNodeToCheckSplittingEnabled == null) {
+            callNodeToCheckSplittingEnabled = DirectCallNode.create(getMethod(arrayClass, "[]").getCallTarget());
+            callNodeToCheckSplittingEnabled.cloneCallTarget();
+        }
+
+        return callNodeToCheckSplittingEnabled.isCallTargetCloned();
+    }
+
     private static final String POST_BOOT_FILE = "/post-boot/post-boot.rb";
 
     public static final String[] CORE_FILES = {

src/main/java/org/truffleruby/core/VMPrimitiveNodes.java

@@ -674,4 +674,14 @@ boolean singleContext() {
         }
     }
 
+    @Primitive(name = "vm_splitting_enabled?")
+    public abstract static class VMSplittingEnabledNode extends PrimitiveArrayArgumentsNode {
+
+        @TruffleBoundary
+        @Specialization
+        boolean isSplittingEnabled() {
+            return getContext().getCoreLibrary().isSplittingEnabled();
+        }
+    }
+
 }