[Truffle] The new Hash#values brings some other stuff onto the fast p…

…ath.
jruby · Mar 4, 2015 · 8bdf5bc · 8bdf5bc
1 parent d282630
commit 8bdf5bc
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 14 deletions.
diff --git a/truffle/src/main/java/org/jruby/truffle/nodes/core/HashNodes.java b/truffle/src/main/java/org/jruby/truffle/nodes/core/HashNodes.java
@@ -555,16 +555,12 @@ public EachNode(EachNode prev) {
 
         @Specialization(guards = "isNull")
         public RubyHash eachNull(RubyHash hash, RubyProc block) {
-            notDesignedForCompilation();
-
             return hash;
         }
 
         @ExplodeLoop
         @Specialization(guards = {"!isNull", "!isBuckets"})
         public RubyHash eachPackedArray(VirtualFrame frame, RubyHash hash, RubyProc block) {
-            notDesignedForCompilation();
-
             final Object[] store = (Object[]) hash.getStore();
             final int size = hash.getSize();
 

diff --git a/truffle/src/main/java/org/jruby/truffle/nodes/core/KernelNodes.java b/truffle/src/main/java/org/jruby/truffle/nodes/core/KernelNodes.java
@@ -1009,21 +1009,19 @@ public InstanceVariableSetNode(RubyContext context, SourceSection sourceSection)
         public InstanceVariableSetNode(InstanceVariableSetNode prev) {
             super(prev);
         }
+
+        // TODO CS 4-Mar-15 this badly needs to be cached
 
+        @TruffleBoundary
         @Specialization
-        public Object isInstanceVariableSet(RubyBasicObject object, RubyString name, Object value) {
-            notDesignedForCompilation();
-
-            notDesignedForCompilation();
+        public Object instanceVariableSet(RubyBasicObject object, RubyString name, Object value) {
             object.getOperations().setInstanceVariable(object, RubyContext.checkInstanceVariableName(getContext(), name.toString(), this), value);
             return value;
         }
 
+        @TruffleBoundary
         @Specialization
-        public Object isInstanceVariableSet(RubyBasicObject object, RubySymbol name, Object value) {
-            notDesignedForCompilation();
-
-            notDesignedForCompilation();
+        public Object instanceVariableSet(RubyBasicObject object, RubySymbol name, Object value) {
             object.getOperations().setInstanceVariable(object, RubyContext.checkInstanceVariableName(getContext(), name.toString(), this), value);
             return value;
         }

diff --git a/truffle/src/main/java/org/jruby/truffle/nodes/core/SymbolNodes.java b/truffle/src/main/java/org/jruby/truffle/nodes/core/SymbolNodes.java
@@ -295,8 +295,6 @@ public ToSNode(ToSNode prev) {
 
         @Specialization
         public RubyString toS(RubySymbol symbol) {
-            notDesignedForCompilation();
-
             return getContext().makeString(symbol.getSymbolBytes().dup());
         }