[Truffle] Use RubyBaseNode to avoid having to write a dummy execute m…

…ethod.
jruby · Apr 16, 2016 · 95f825a · 95f825a · chrisseaton · Apr 16, 2016
1 parent 343718d
commit 95f825a
Showing 4 changed files with 8 additions and 23 deletions.
diff --git a/truffle/src/main/java/org/jruby/truffle/core/hash/HashNode.java b/truffle/src/main/java/org/jruby/truffle/core/hash/HashNode.java
@@ -13,11 +13,12 @@
 import com.oracle.truffle.api.profiles.ConditionProfile;
 import com.oracle.truffle.api.source.SourceSection;
 import org.jruby.truffle.RubyContext;
+import org.jruby.truffle.language.RubyBaseNode;
 import org.jruby.truffle.language.RubyNode;
 import org.jruby.truffle.language.dispatch.CallDispatchHeadNode;
 import org.jruby.truffle.language.dispatch.DispatchHeadNodeFactory;
 
-public class HashNode extends RubyNode {
+public class HashNode extends RubyBaseNode {
 
     @Child private CallDispatchHeadNode hashNode;
 
@@ -41,9 +42,4 @@ public int hash(VirtualFrame frame, Object key) {
         }
     }
 
-    @Override
-    public Object execute(VirtualFrame frame) {
-        throw new UnsupportedOperationException();
-    }
-
 }
diff --git a/truffle/src/main/java/org/jruby/truffle/core/hash/LookupEntryNode.java b/truffle/src/main/java/org/jruby/truffle/core/hash/LookupEntryNode.java
@@ -17,11 +17,12 @@
 import org.jruby.truffle.core.Layouts;
 import org.jruby.truffle.core.basicobject.BasicObjectNodes;
 import org.jruby.truffle.core.basicobject.BasicObjectNodesFactory;
+import org.jruby.truffle.language.RubyBaseNode;
 import org.jruby.truffle.language.RubyNode;
 import org.jruby.truffle.language.dispatch.CallDispatchHeadNode;
 import org.jruby.truffle.language.dispatch.DispatchHeadNodeFactory;
 
-public class LookupEntryNode extends RubyNode {
+public class LookupEntryNode extends RubyBaseNode {
 
     @Child HashNode hashNode;
     @Child CallDispatchHeadNode eqlNode;
@@ -63,9 +64,4 @@ public HashLookupResult lookup(VirtualFrame frame, DynamicObject hash, Object ke
         return new HashLookupResult(hashed, index, previousEntry, null);
     }
 
-    @Override
-    public Object execute(VirtualFrame frame) {
-        throw new UnsupportedOperationException();
-    }
-
 }
diff --git a/truffle/src/main/java/org/jruby/truffle/core/numeric/FixnumOrBignumNode.java b/truffle/src/main/java/org/jruby/truffle/core/numeric/FixnumOrBignumNode.java
@@ -17,12 +17,13 @@
 import org.jruby.truffle.RubyContext;
 import org.jruby.truffle.core.CoreLibrary;
 import org.jruby.truffle.core.Layouts;
+import org.jruby.truffle.language.RubyBaseNode;
 import org.jruby.truffle.language.RubyNode;
 
 import java.math.BigDecimal;
 import java.math.BigInteger;
 
-public class FixnumOrBignumNode extends RubyNode {
+public class FixnumOrBignumNode extends RubyBaseNode {
 
     private static final BigInteger LONG_MIN_BIGINT = BigInteger.valueOf(Long.MIN_VALUE);
     private static final BigInteger LONG_MAX_BIGINT = BigInteger.valueOf(Long.MAX_VALUE);
@@ -76,8 +77,4 @@ private static BigInteger doubleToBigInteger(double value) {
         return new BigDecimal(value).toBigInteger();
     }
 
-    public Object execute(VirtualFrame frame) {
-        throw new UnsupportedOperationException();
-    }
-
 }
diff --git a/truffle/src/main/java/org/jruby/truffle/core/numeric/GeneralDivModNode.java b/truffle/src/main/java/org/jruby/truffle/core/numeric/GeneralDivModNode.java
@@ -16,12 +16,13 @@
 import com.oracle.truffle.api.source.SourceSection;
 import org.jruby.truffle.RubyContext;
 import org.jruby.truffle.core.Layouts;
+import org.jruby.truffle.language.RubyBaseNode;
 import org.jruby.truffle.language.RubyNode;
 import org.jruby.truffle.language.control.RaiseException;
 
 import java.math.BigInteger;
 
-public class GeneralDivModNode extends RubyNode {
+public class GeneralDivModNode extends RubyBaseNode {
 
     @Child private FixnumOrBignumNode fixnumOrBignumQuotient;
     @Child private FixnumOrBignumNode fixnumOrBignumRemainder;
@@ -172,9 +173,4 @@ public DynamicObject create(BigInteger value) {
         return Layouts.BIGNUM.createBignum(coreLibrary().getBignumFactory(), value);
     }
 
-    @Override
-    public Object execute(VirtualFrame frame) {
-        throw new UnsupportedOperationException();
-    }
-
 }