jruby · Dec 3, 2015 · Dec 3, 2015 · Dec 3, 2015 · Dec 3, 2015 · Dec 3, 2015
diff --git a/spec/truffle/specs/truffle/fixnum/and_spec.rb b/spec/truffle/specs/truffle/fixnum/and_spec.rb
@@ -11,7 +11,7 @@
 describe "Fixnum#&" do
   before :each do
     @long = (1 << 48) + 1
-    @mask = (1 << 30) - 1
+    @mask = Truffle::Primitive.fixnum_lower((1 << 30) - 1)
   end
 
   it "returns an int for (int, int)" do

diff --git a/truffle/src/main/java/org/jruby/truffle/nodes/core/fixnum/FixnumNodes.java b/truffle/src/main/java/org/jruby/truffle/nodes/core/fixnum/FixnumNodes.java
@@ -12,12 +12,15 @@
 import com.oracle.truffle.api.CompilerDirectives;
 import com.oracle.truffle.api.CompilerDirectives.TruffleBoundary;
 import com.oracle.truffle.api.ExactMath;
+import com.oracle.truffle.api.dsl.Cached;
 import com.oracle.truffle.api.dsl.Specialization;
 import com.oracle.truffle.api.frame.VirtualFrame;
 import com.oracle.truffle.api.nodes.UnexpectedResultException;
 import com.oracle.truffle.api.object.DynamicObject;
 import com.oracle.truffle.api.source.SourceSection;
 import com.oracle.truffle.api.utilities.BranchProfile;
+import com.oracle.truffle.api.utilities.ConditionProfile;
+
 import org.jcodings.specific.USASCIIEncoding;
 import org.jruby.truffle.nodes.RubyNode;
 import org.jruby.truffle.nodes.core.*;
@@ -28,6 +31,7 @@
 import org.jruby.truffle.runtime.NotProvided;
 import org.jruby.truffle.runtime.RubyContext;
 import org.jruby.truffle.runtime.control.RaiseException;
+import org.jruby.truffle.runtime.core.CoreLibrary;
 import org.jruby.truffle.runtime.core.StringOperations;
 import org.jruby.truffle.runtime.layouts.Layouts;
 import org.jruby.util.StringSupport;
@@ -770,7 +774,7 @@ public int bitAndLongInt(long a, int b) {
             return (int) (a & b);
         }
 
-        @Specialization(contains = { "bitAndIntInt", "bitAndIntLong", "bitAndLongInt" })
+        @Specialization
         public long bitAndLongLong(long a, long b) {
             return a & b;
         }
@@ -833,7 +837,7 @@ public Object bitXOr(VirtualFrame frame, Object a, DynamicObject b) {
 
     }
 
-    @CoreMethod(names = "<<", required = 1, lowerFixnumParameters = { 0, 1 })
+    @CoreMethod(names = "<<", required = 1, lowerFixnumParameters = 0)
     public abstract static class LeftShiftNode extends BignumNodes.BignumCoreMethodNode {
 
         @Child private RightShiftNode rightShiftNode;
@@ -850,6 +854,11 @@ public int leftShift(int a, int b) {
             return a << b;
         }
 
+        @Specialization(guards = { "b >= 0", "canShiftLongIntoInt(a, b)" })
+        public int leftShift(long a, int b) {
+            return (int) (a << b);
+        }
+
         @Specialization(guards = { "b >= 0", "canShiftIntoLong(a, b)" })
         public long leftShiftToLong(long a, int b) {
             return a << b;
@@ -886,13 +895,17 @@ static boolean canShiftIntoInt(int a, int b) {
             return Integer.numberOfLeadingZeros(a) - b > 0;
         }
 
+        static boolean canShiftLongIntoInt(long a, int b) {
+            return Long.numberOfLeadingZeros(a) - 32 - b > 0;
+        }
+
         static boolean canShiftIntoLong(long a, int b) {
             return Long.numberOfLeadingZeros(a) - b > 0;
         }
 
     }
 
-    @CoreMethod(names = ">>", required = 1, lowerFixnumParameters = { 0, 1 })
+    @CoreMethod(names = ">>", required = 1, lowerFixnumParameters = 0)
     public abstract static class RightShiftNode extends CoreMethodArrayArgumentsNode {
 
         @Child private CallDispatchHeadNode fallbackCallNode;
@@ -905,20 +918,22 @@ public RightShiftNode(RubyContext context, SourceSection sourceSection) {
         public abstract Object executeRightShift(VirtualFrame frame, Object a, Object b);
 
         @Specialization(guards = "b >= 0")
-        public int rightShift(VirtualFrame frame, int a, int b) {
-            if (b >= Integer.SIZE - 1) {
+        public int rightShift(VirtualFrame frame, int a, int b,
+                @Cached("createBinaryProfile()") ConditionProfile profile) {
+            if (profile.profile(b >= Integer.SIZE - 1)) {
                 return a < 0 ? -1 : 0;
             } else {
                 return a >> b;
             }
         }
 
         @Specialization(guards = "b >= 0")
-        public long rightShift(VirtualFrame frame, long a, int b) {
-            if (b >= Long.SIZE - 1) {
-                return a < 0 ? -1 : 0;
+        public Object rightShift(VirtualFrame frame, long a, int b,
+                @Cached("createBinaryProfile()") ConditionProfile profile) {
+            if (profile.profile(b >= Long.SIZE - 1)) {
+                return a < 0 ? -1 : 0; // int
             } else {
-                return a >> b;
+                return a >> b; // long
             }
         }
 
@@ -933,6 +948,7 @@ public Object rightShiftNeg(VirtualFrame frame, long a, int b) {
 
         @Specialization(guards = "b >= 0")
         public int rightShift(long a, long b) { // b is not in int range due to lowerFixnumParameters
+            assert !CoreLibrary.fitsIntoInteger(b);
             return 0;
         }
 

diff --git a/truffle/src/main/java/org/jruby/truffle/nodes/rubinius/IOPrimitiveNodes.java b/truffle/src/main/java/org/jruby/truffle/nodes/rubinius/IOPrimitiveNodes.java
@@ -149,7 +149,7 @@ private void newOpenFd(int newFd) {
         }
     }
 
-    @RubiniusPrimitive(name = "io_open", needsSelf = false)
+    @RubiniusPrimitive(name = "io_open", needsSelf = false, lowerFixnumParameters = { 1, 2 })
     public static abstract class IOOpenPrimitiveNode extends RubiniusPrimitiveNode {
 
         public IOOpenPrimitiveNode(RubyContext context, SourceSection sourceSection) {
@@ -171,11 +171,6 @@ public IOTruncatePrimitiveNode(RubyContext context, SourceSection sourceSection)
             super(context, sourceSection);
         }
 
-        @Specialization(guards = "isRubyString(path)")
-        public int truncate(DynamicObject path, int length) {
-            return truncate(path, (long) length);
-        }
-
         @Specialization(guards = "isRubyString(path)")
         public int truncate(DynamicObject path, long length) {
             final int result = posix().truncate(StringOperations.getString(getContext(), path), length);
@@ -195,11 +190,6 @@ public IOFTruncatePrimitiveNode(RubyContext context, SourceSection sourceSection
             super(context, sourceSection);
         }
 
-        @Specialization
-        public int ftruncate(VirtualFrame frame, DynamicObject io, int length) {
-            return ftruncate(frame, io, (long) length);
-        }
-
         @Specialization
         public int ftruncate(VirtualFrame frame, DynamicObject io, long length) {
             final int fd = Layouts.IO.getDescriptor(io);
@@ -346,7 +336,7 @@ public Object reopen(VirtualFrame frame, DynamicObject file, DynamicObject io) {
 
     }
 
-    @RubiniusPrimitive(name = "io_reopen_path")
+    @RubiniusPrimitive(name = "io_reopen_path", lowerFixnumParameters = 1)
     public static abstract class IOReopenPathPrimitiveNode extends RubiniusPrimitiveNode {
 
         @Child private CallDispatchHeadNode resetBufferingNode;

diff --git a/truffle/src/main/java/org/jruby/truffle/nodes/rubinius/RegexpPrimitiveNodes.java b/truffle/src/main/java/org/jruby/truffle/nodes/rubinius/RegexpPrimitiveNodes.java
@@ -47,7 +47,7 @@ public boolean fixedEncoding(DynamicObject regexp) {
 
     }
 
-    @RubiniusPrimitive(name = "regexp_initialize")
+    @RubiniusPrimitive(name = "regexp_initialize", lowerFixnumParameters = 1)
     @ImportStatic(RegexpGuards.class)
     public static abstract class RegexpInitializePrimitiveNode extends RubiniusPrimitiveNode {