[Truffle] We know the block argument is a block.

* Because calling with 1.times(&b) will call #to_proc and ToProcCastNode should check the return value.
jruby · Aug 28, 2015 · 6200f1a · 6200f1a
1 parent 2b6b870
commit 6200f1a
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/truffle/src/main/java/org/jruby/truffle/nodes/core/IntegerNodes.java b/truffle/src/main/java/org/jruby/truffle/nodes/core/IntegerNodes.java
@@ -103,7 +103,7 @@ public DynamicObject times(VirtualFrame frame, int n, NotProvided block) {
             return Layouts.ARRAY.createArray(getContext().getCoreLibrary().getArrayFactory(), array, n);
         }
 
-        @Specialization(guards = "isRubyProc(block)")
+        @Specialization
         public Object times(VirtualFrame frame, int n, DynamicObject block) {
             int count = 0;
 
@@ -124,7 +124,7 @@ public Object times(VirtualFrame frame, int n, DynamicObject block) {
             return n;
         }
 
-        @Specialization(guards = "isRubyProc(block)")
+        @Specialization
         public Object times(VirtualFrame frame, long n, DynamicObject block) {
             int count = 0;
 
@@ -145,7 +145,7 @@ public Object times(VirtualFrame frame, long n, DynamicObject block) {
             return n;
         }
 
-        @Specialization(guards = {"isRubyBignum(n)", "isRubyProc(block)"})
+        @Specialization(guards = "isRubyBignum(n)")
         public Object times(VirtualFrame frame, DynamicObject n, DynamicObject block,
                 @Cached("create(getContext(), getSourceSection())") FixnumOrBignumNode fixnumOrBignumNode) {