Revert "[Truffle] Use declarative guards for checking classes in the IC."

This reverts commit f2b1582.

Author: chrisseaton

core/src/main/java/org/jruby/truffle/nodes/dispatch/CachedBoxedDispatchNode.java

@@ -103,13 +103,24 @@ public CachedBoxedDispatchNode(CachedBoxedDispatchNode prev) {
         indirectCallNode = prev.indirectCallNode;
     }
 
-    @Specialization(guards = {"guardClass", "guardName"})
+    @Specialization(guards = "guardName")
     public Object dispatch(
             VirtualFrame frame,
             RubyBasicObject receiverObject,
             Object methodName,
             Object blockObject,
             Object argumentsObjects) {
+        // Check the lookup node is what we expect
+
+        if (receiverObject.getMetaClass() != expectedClass) {
+            return next.executeDispatch(
+                    frame,
+                    receiverObject,
+                    methodName,
+                    CompilerDirectives.unsafeCast(blockObject, RubyProc.class, true, false),
+                    argumentsObjects);
+        }
+
         // Check the class has not been modified
 
         try {
@@ -159,14 +170,6 @@ public Object dispatch(
         }
     }
 
-    protected final boolean guardClass(
-            RubyBasicObject receiverObject,
-            Object methodName,
-            Object blockObject,
-            Object argumentsObjects) {
-        return receiverObject.getMetaClass() == expectedClass;
-    }
-
     @Fallback
     public Object dispatch(
             VirtualFrame frame,

core/src/main/java/org/jruby/truffle/nodes/dispatch/CachedBoxedMethodMissingDispatchNode.java

@@ -83,13 +83,24 @@ public CachedBoxedMethodMissingDispatchNode(CachedBoxedMethodMissingDispatchNode
         indirectCallNode = prev.indirectCallNode;
     }
 
-    @Specialization(guards = {"guardClass", "guardName"})
+    @Specialization(guards = "guardName")
     public Object dispatch(
             VirtualFrame frame,
             RubyBasicObject receiverObject,
             Object methodName,
             Object blockObject,
             Object argumentsObjects) {
+        // Check the lookup node is what we expect
+
+        if (receiverObject.getMetaClass() != expectedClass) {
+            return next.executeDispatch(
+                    frame,
+                    receiverObject,
+                    methodName,
+                    CompilerDirectives.unsafeCast(blockObject, RubyProc.class, true, false),
+                    argumentsObjects);
+        }
+
         // Check the class has not been modified
 
         try {
@@ -166,14 +177,6 @@ public Object dispatch(
         }
     }
 
-    protected final boolean guardClass(
-            RubyBasicObject receiverObject,
-            Object methodName,
-            Object blockObject,
-            Object argumentsObjects) {
-        return receiverObject.getMetaClass() == expectedClass;
-    }
-
     @Fallback
     public Object dispatch(
             VirtualFrame frame,

core/src/main/java/org/jruby/truffle/nodes/dispatch/CachedBoxedReturnMissingDispatchNode.java

@@ -45,13 +45,24 @@ public CachedBoxedReturnMissingDispatchNode(CachedBoxedReturnMissingDispatchNode
         unmodifiedAssumption = prev.unmodifiedAssumption;
     }
 
-    @Specialization(guards = {"guardClass", "guardName"})
+    @Specialization(guards = "guardName")
     public Object dispatch(
             VirtualFrame frame,
             RubyBasicObject receiverObject,
             Object methodName,
             Object blockObject,
             Object argumentsObjects) {
+        // Check the lookup node is what we expect
+
+        if (receiverObject.getMetaClass() != expectedClass) {
+            return next.executeDispatch(
+                    frame,
+                    receiverObject,
+                    methodName,
+                    CompilerDirectives.unsafeCast(blockObject, RubyProc.class, true, false),
+                    argumentsObjects);
+        }
+
         // Check the class has not been modified
 
         try {
@@ -78,14 +89,6 @@ public Object dispatch(
         }
     }
 
-    protected final boolean guardClass(
-            RubyBasicObject receiverObject,
-            Object methodName,
-            Object blockObject,
-            Object argumentsObjects) {
-        return receiverObject.getMetaClass() == expectedClass;
-    }
-
     @Fallback
     public Object dispatch(
             VirtualFrame frame,

core/src/main/java/org/jruby/truffle/nodes/dispatch/CachedUnboxedDispatchNode.java

@@ -69,13 +69,24 @@ public CachedUnboxedDispatchNode(CachedUnboxedDispatchNode prev) {
         indirectCallNode = prev.indirectCallNode;
     }
 
-    @Specialization(guards = {"!isRubyBasicObject", "guardClass", "guardName"})
+    @Specialization(guards = {"!isRubyBasicObject", "guardName"})
     public Object dispatch(
             VirtualFrame frame,
             Object receiverObject,
             Object methodName,
             Object blockObject,
             Object argumentsObjects) {
+        // Check the class is what we expect
+
+        if (receiverObject.getClass() != expectedClass) {
+            return next.executeDispatch(
+                    frame,
+                    receiverObject,
+                    methodName,
+                    blockObject,
+                    argumentsObjects);
+        }
+
         // Check the class has not been modified
 
         try {
@@ -123,14 +134,6 @@ public Object dispatch(
         }
     }
 
-    protected final boolean guardClass(
-            Object receiverObject,
-            Object methodName,
-            Object blockObject,
-            Object argumentsObjects) {
-        return receiverObject.getClass() == expectedClass;
-    }
-
     @Fallback
     public Object fallback(
             VirtualFrame frame,