Steps towards unification of number bitset and range types.

src/compiler/change-lowering.cc

Index: src/compiler/change-lowering.cc
diff --git a/src/compiler/change-lowering.cc b/src/compiler/change-lowering.cc
index 7ddc751ab53199af0e8fc501b6f7eb12215f0d3c..bb1d79b492d7c8e0392c38cb01d5e7c184aa9864 100644
--- a/src/compiler/change-lowering.cc
+++ b/src/compiler/change-lowering.cc
@@ -163,7 +163,7 @@ Reduction ChangeLowering::ChangeInt32ToTagged(Node* value, Node* control) {
         machine()->Word64Shl(),
         graph()->NewNode(machine()->ChangeInt32ToInt64(), value),
         SmiShiftBitsConstant()));
-  } else if (NodeProperties::GetBounds(value).upper->Is(Type::SignedSmall())) {
+  } else if (NodeProperties::GetBounds(value).upper->Is(Type::Signed31())) {

[rossberg, 2014/12/12 13:57:50]

I think this should still refer to SignedSmall; Signed31 maybe shouldn't even be
exposed.

[Jarin, 2015/01/08 14:30:26]

Interestingly, we test this even on 64-bit architecture, and those tests broke
without changing SignedSmall to Signed31 here.

If you insist, I could change this, but then we would have to disable the test,
too.

     return Replace(
         graph()->NewNode(machine()->WordShl(), value, SmiShiftBitsConstant()));
   }