Remove obsolete kNumber binop feedback.

With the removal of Crankshaft, kNumber has become obsolete as
BinaryOperationFeedback. Turbofan uses kNumberOrOddball.

Bug: 
Change-Id: If577f5efcc81d7c08f43908f2764ff0ec6f8747c
Reviewed-on: https://chromium-review.googlesource.com/628376Reviewed-by: Jaroslav Sevcik <jarin@chromium.org>
Reviewed-by: Mythri Alle <mythria@chromium.org>
Reviewed-by: Jakob Kummerow <jkummerow@chromium.org>
Commit-Queue: Jakob Kummerow <jkummerow@chromium.org>
Cr-Commit-Position: refs/heads/master@{#47555}

src/compiler/js-operator.cc

@@ -602,7 +602,6 @@ struct JSOperatorGlobalCache final {
   Name##Operator<BinaryOperationHint::kSignedSmallInputs>                     \
       k##Name##SignedSmallInputsOperator;                                     \
   Name##Operator<BinaryOperationHint::kSigned32> k##Name##Signed32Operator;   \
-  Name##Operator<BinaryOperationHint::kNumber> k##Name##NumberOperator;       \
   Name##Operator<BinaryOperationHint::kNumberOrOddball>                       \
       k##Name##NumberOrOddballOperator;                                       \
   Name##Operator<BinaryOperationHint::kString> k##Name##StringOperator;       \
@@ -658,8 +657,6 @@ CACHED_OP_LIST(CACHED_OP)
         return &cache_.k##Name##SignedSmallInputsOperator;            \
       case BinaryOperationHint::kSigned32:                            \
         return &cache_.k##Name##Signed32Operator;                     \
-      case BinaryOperationHint::kNumber:                              \
-        return &cache_.k##Name##NumberOperator;                       \
       case BinaryOperationHint::kNumberOrOddball:                     \
         return &cache_.k##Name##NumberOrOddballOperator;              \
       case BinaryOperationHint::kString:                              \

src/compiler/js-type-hint-lowering.cc

@@ -28,9 +28,6 @@ bool BinaryOperationHintToNumberOperationHint(
     case BinaryOperationHint::kSigned32:
       *number_hint = NumberOperationHint::kSigned32;
       return true;
-    case BinaryOperationHint::kNumber:
-      *number_hint = NumberOperationHint::kNumber;
-      return true;
     case BinaryOperationHint::kNumberOrOddball:
       *number_hint = NumberOperationHint::kNumberOrOddball;
       return true;

src/feedback-vector-inl.h

@@ -183,8 +183,6 @@ BinaryOperationHint BinaryOperationHintFromFeedback(int type_feedback) {
       return BinaryOperationHint::kSignedSmall;
     case BinaryOperationFeedback::kSignedSmallInputs:
       return BinaryOperationHint::kSignedSmallInputs;
-    case BinaryOperationFeedback::kNumber:
-      return BinaryOperationHint::kNumber;
     case BinaryOperationFeedback::kNumberOrOddball:
       return BinaryOperationHint::kNumberOrOddball;
     case BinaryOperationFeedback::kString:

src/globals.h

@@ -1254,23 +1254,17 @@ inline uint32_t ObjectHash(Address address) {
 // Type feedback is encoded in such a way that, we can combine the feedback
 // at different points by performing an 'OR' operation. Type feedback moves
 // to a more generic type when we combine feedback.
-// kSignedSmall -> kSignedSmallInputs -> kNumber  -> kNumberOrOddball -> kAny
-//                                                   kString          -> kAny
-// TODO(mythria): Remove kNumber type when crankshaft can handle Oddballs
-// similar to Numbers. We don't need kNumber feedback for Turbofan. Extra
-// information about Number might reduce few instructions but causes more
-// deopts. We collect Number only because crankshaft does not handle all
-// cases of oddballs.
+// kSignedSmall -> kSignedSmallInputs -> kNumberOrOddball -> kAny
+//                                       kString          -> kAny
 class BinaryOperationFeedback {
  public:
   enum {
     kNone = 0x0,
     kSignedSmall = 0x1,
     kSignedSmallInputs = 0x3,
-    kNumber = 0x7,
-    kNumberOrOddball = 0xF,
-    kString = 0x10,
-    kAny = 0x3F
+    kNumberOrOddball = 0x5,
+    kString = 0x8,
+    kAny = 0x1F
   };
 };
 
@@ -1281,7 +1275,6 @@ class BinaryOperationFeedback {
 // kInternalizedString -> kString   -> kAny
 //                        kSymbol   -> kAny
 //                        kReceiver -> kAny
-// TODO(epertoso): consider unifying this with BinaryOperationFeedback.
 class CompareOperationFeedback {
  public:
   enum {

src/ic/binary-op-assembler.cc

@@ -122,7 +122,8 @@ Node* BinaryOpAssembler::Generate_AddWithFeedback(Node* context, Node* lhs,
 
   BIND(&do_fadd);
   {
-    var_type_feedback.Bind(SmiConstant(BinaryOperationFeedback::kNumber));
+    var_type_feedback.Bind(
+        SmiConstant(BinaryOperationFeedback::kNumberOrOddball));
     Node* value = Float64Add(var_fadd_lhs.value(), var_fadd_rhs.value());
     Node* result = AllocateHeapNumberWithValue(value);
     var_result.Bind(result);
@@ -293,7 +294,8 @@ Node* BinaryOpAssembler::Generate_BinaryOperationWithFeedback(
 
   BIND(&do_float_operation);
   {
-    var_type_feedback.Bind(SmiConstant(BinaryOperationFeedback::kNumber));
+    var_type_feedback.Bind(
+        SmiConstant(BinaryOperationFeedback::kNumberOrOddball));
     Node* lhs_value = var_float_lhs.value();
     Node* rhs_value = var_float_rhs.value();
     Node* value = floatOperation(lhs_value, rhs_value);
@@ -410,7 +412,8 @@ Node* BinaryOpAssembler::Generate_SubtractWithFeedback(Node* context, Node* lhs,
 
     BIND(&if_overflow);
     {
-      var_type_feedback->Bind(SmiConstant(BinaryOperationFeedback::kNumber));
+      var_type_feedback->Bind(
+          SmiConstant(BinaryOperationFeedback::kNumberOrOddball));
       Node* value = Float64Sub(SmiToFloat64(lhs), SmiToFloat64(rhs));
       var_result.Bind(AllocateHeapNumberWithValue(value));
       Goto(&end);
@@ -436,7 +439,7 @@ Node* BinaryOpAssembler::Generate_MultiplyWithFeedback(Node* context, Node* lhs,
     Node* result = SmiMul(lhs, rhs);
     var_type_feedback->Bind(SelectSmiConstant(
         TaggedIsSmi(result), BinaryOperationFeedback::kSignedSmall,
-        BinaryOperationFeedback::kNumber));
+        BinaryOperationFeedback::kNumberOrOddball));
     return result;
   };
   auto floatFunction = [=](Node* lhs, Node* rhs) {
@@ -488,7 +491,7 @@ Node* BinaryOpAssembler::Generate_ModulusWithFeedback(
     Node* result = SmiMod(lhs, rhs);
     var_type_feedback->Bind(SelectSmiConstant(
         TaggedIsSmi(result), BinaryOperationFeedback::kSignedSmall,
-        BinaryOperationFeedback::kNumber));
+        BinaryOperationFeedback::kNumberOrOddball));
     return result;
   };
   auto floatFunction = [=](Node* lhs, Node* rhs) {

src/interpreter/interpreter-assembler.cc

@@ -1191,7 +1191,7 @@ Node* InterpreterAssembler::TruncateTaggedToWord32WithFeedback(
         var_result.Bind(TruncateHeapNumberValueToWord32(value));
         var_type_feedback->Bind(
             SmiOr(var_type_feedback->value(),
-                  SmiConstant(BinaryOperationFeedback::kNumber)));
+                  SmiConstant(BinaryOperationFeedback::kNumberOrOddball)));
         Goto(&done_loop);
       }
 

src/interpreter/interpreter-generator.cc

@@ -1017,9 +1017,9 @@ class InterpreterBitwiseBinaryOpAssembler : public InterpreterAssembler {
         UNREACHABLE();
     }
 
-    Node* result_type = SelectSmiConstant(TaggedIsSmi(result),
-                                          BinaryOperationFeedback::kSignedSmall,
-                                          BinaryOperationFeedback::kNumber);
+    Node* result_type = SelectSmiConstant(
+        TaggedIsSmi(result), BinaryOperationFeedback::kSignedSmall,
+        BinaryOperationFeedback::kNumberOrOddball);
 
     if (FLAG_debug_code) {
       Label ok(this);
@@ -1108,9 +1108,9 @@ IGNITION_HANDLER(BitwiseOrSmi, InterpreterAssembler) {
   Node* rhs_value = SmiToWord32(right);
   Node* value = Word32Or(lhs_value, rhs_value);
   Node* result = ChangeInt32ToTagged(value);
-  Node* result_type = SelectSmiConstant(TaggedIsSmi(result),
-                                        BinaryOperationFeedback::kSignedSmall,
-                                        BinaryOperationFeedback::kNumber);
+  Node* result_type = SelectSmiConstant(
+      TaggedIsSmi(result), BinaryOperationFeedback::kSignedSmall,
+      BinaryOperationFeedback::kNumberOrOddball);
   Node* function = LoadRegister(Register::function_closure());
   UpdateFeedback(SmiOr(result_type, var_lhs_type_feedback.value()),
                  feedback_vector, slot_index, function);
@@ -1134,9 +1134,9 @@ IGNITION_HANDLER(BitwiseXorSmi, InterpreterAssembler) {
   Node* rhs_value = SmiToWord32(right);
   Node* value = Word32Xor(lhs_value, rhs_value);
   Node* result = ChangeInt32ToTagged(value);
-  Node* result_type = SelectSmiConstant(TaggedIsSmi(result),
-                                        BinaryOperationFeedback::kSignedSmall,
-                                        BinaryOperationFeedback::kNumber);
+  Node* result_type = SelectSmiConstant(
+      TaggedIsSmi(result), BinaryOperationFeedback::kSignedSmall,
+      BinaryOperationFeedback::kNumberOrOddball);
   Node* function = LoadRegister(Register::function_closure());
   UpdateFeedback(SmiOr(result_type, var_lhs_type_feedback.value()),
                  feedback_vector, slot_index, function);
@@ -1160,9 +1160,9 @@ IGNITION_HANDLER(BitwiseAndSmi, InterpreterAssembler) {
   Node* rhs_value = SmiToWord32(right);
   Node* value = Word32And(lhs_value, rhs_value);
   Node* result = ChangeInt32ToTagged(value);
-  Node* result_type = SelectSmiConstant(TaggedIsSmi(result),
-                                        BinaryOperationFeedback::kSignedSmall,
-                                        BinaryOperationFeedback::kNumber);
+  Node* result_type = SelectSmiConstant(
+      TaggedIsSmi(result), BinaryOperationFeedback::kSignedSmall,
+      BinaryOperationFeedback::kNumberOrOddball);
   Node* function = LoadRegister(Register::function_closure());
   UpdateFeedback(SmiOr(result_type, var_lhs_type_feedback.value()),
                  feedback_vector, slot_index, function);
@@ -1189,9 +1189,9 @@ IGNITION_HANDLER(ShiftLeftSmi, InterpreterAssembler) {
   Node* shift_count = Word32And(rhs_value, Int32Constant(0x1f));
   Node* value = Word32Shl(lhs_value, shift_count);
   Node* result = ChangeInt32ToTagged(value);
-  Node* result_type = SelectSmiConstant(TaggedIsSmi(result),
-                                        BinaryOperationFeedback::kSignedSmall,
-                                        BinaryOperationFeedback::kNumber);
+  Node* result_type = SelectSmiConstant(
+      TaggedIsSmi(result), BinaryOperationFeedback::kSignedSmall,
+      BinaryOperationFeedback::kNumberOrOddball);
   Node* function = LoadRegister(Register::function_closure());
   UpdateFeedback(SmiOr(result_type, var_lhs_type_feedback.value()),
                  feedback_vector, slot_index, function);
@@ -1218,9 +1218,9 @@ IGNITION_HANDLER(ShiftRightSmi, InterpreterAssembler) {
   Node* shift_count = Word32And(rhs_value, Int32Constant(0x1f));
   Node* value = Word32Sar(lhs_value, shift_count);
   Node* result = ChangeInt32ToTagged(value);
-  Node* result_type = SelectSmiConstant(TaggedIsSmi(result),
-                                        BinaryOperationFeedback::kSignedSmall,
-                                        BinaryOperationFeedback::kNumber);
+  Node* result_type = SelectSmiConstant(
+      TaggedIsSmi(result), BinaryOperationFeedback::kSignedSmall,
+      BinaryOperationFeedback::kNumberOrOddball);
   Node* function = LoadRegister(Register::function_closure());
   UpdateFeedback(SmiOr(result_type, var_lhs_type_feedback.value()),
                  feedback_vector, slot_index, function);
@@ -1247,9 +1247,9 @@ IGNITION_HANDLER(ShiftRightLogicalSmi, InterpreterAssembler) {
   Node* shift_count = Word32And(rhs_value, Int32Constant(0x1f));
   Node* value = Word32Shr(lhs_value, shift_count);
   Node* result = ChangeUint32ToTagged(value);
-  Node* result_type = SelectSmiConstant(TaggedIsSmi(result),
-                                        BinaryOperationFeedback::kSignedSmall,
-                                        BinaryOperationFeedback::kNumber);
+  Node* result_type = SelectSmiConstant(
+      TaggedIsSmi(result), BinaryOperationFeedback::kSignedSmall,
+      BinaryOperationFeedback::kNumberOrOddball);
   Node* function = LoadRegister(Register::function_closure());
   UpdateFeedback(SmiOr(result_type, var_lhs_type_feedback.value()),
                  feedback_vector, slot_index, function);
@@ -1294,7 +1294,8 @@ IGNITION_HANDLER(ToNumber, InterpreterAssembler) {
   BIND(&if_objectisnumber);
   {
     var_result.Bind(object);
-    var_type_feedback.Bind(SmiConstant(BinaryOperationFeedback::kNumber));
+    var_type_feedback.Bind(
+        SmiConstant(BinaryOperationFeedback::kNumberOrOddball));
     Goto(&if_done);
   }
 
@@ -1443,7 +1444,7 @@ IGNITION_HANDLER(Inc, InterpreterAssembler) {
     Node* finc_result = Float64Add(finc_value, one);
     var_type_feedback.Bind(
         SmiOr(var_type_feedback.value(),
-              SmiConstant(BinaryOperationFeedback::kNumber)));
+              SmiConstant(BinaryOperationFeedback::kNumberOrOddball)));
     result_var.Bind(AllocateHeapNumberWithValue(finc_result));
     Goto(&end);
   }
@@ -1568,7 +1569,7 @@ IGNITION_HANDLER(Dec, InterpreterAssembler) {
     Node* fdec_result = Float64Sub(fdec_value, one);
     var_type_feedback.Bind(
         SmiOr(var_type_feedback.value(),
-              SmiConstant(BinaryOperationFeedback::kNumber)));
+              SmiConstant(BinaryOperationFeedback::kNumberOrOddball)));
     result_var.Bind(AllocateHeapNumberWithValue(fdec_result));
     Goto(&end);
   }

src/type-hints.cc

@@ -17,8 +17,6 @@ std::ostream& operator<<(std::ostream& os, BinaryOperationHint hint) {
       return os << "SignedSmallInputs";
     case BinaryOperationHint::kSigned32:
       return os << "Signed32";
-    case BinaryOperationHint::kNumber:
-      return os << "Number";
     case BinaryOperationHint::kNumberOrOddball:
       return os << "NumberOrOddball";
     case BinaryOperationHint::kString:

src/type-hints.h

@@ -17,7 +17,6 @@ enum class BinaryOperationHint : uint8_t {
   kSignedSmall,
   kSignedSmallInputs,
   kSigned32,
-  kNumber,
   kNumberOrOddball,
   kString,
   kAny

test/cctest/interpreter/test-interpreter.cc

@@ -559,14 +559,14 @@ TEST(InterpreterBinaryOpTypeFeedback) {
       {Token::Value::ADD, ast_factory.NewSmi(Smi::kMaxValue),
        ast_factory.NewSmi(1),
        isolate->factory()->NewHeapNumber(Smi::kMaxValue + 1.0),
-       BinaryOperationFeedback::kNumber},
+       BinaryOperationFeedback::kNumberOrOddball},
       {Token::Value::ADD, ast_factory.NewNumber(3.1415), ast_factory.NewSmi(3),
        isolate->factory()->NewHeapNumber(3.1415 + 3),
-       BinaryOperationFeedback::kNumber},
+       BinaryOperationFeedback::kNumberOrOddball},
       {Token::Value::ADD, ast_factory.NewNumber(3.1415),
        ast_factory.NewNumber(1.4142),
        isolate->factory()->NewHeapNumber(3.1415 + 1.4142),
-       BinaryOperationFeedback::kNumber},
+       BinaryOperationFeedback::kNumberOrOddball},
       {Token::Value::ADD,
        ast_factory.NewString(ast_factory.GetOneByteString("foo")),
        ast_factory.NewString(ast_factory.GetOneByteString("bar")),
@@ -584,14 +584,14 @@ TEST(InterpreterBinaryOpTypeFeedback) {
        ast_factory.NewSmi(static_cast<uint32_t>(Smi::kMinValue)),
        ast_factory.NewSmi(1),
        isolate->factory()->NewHeapNumber(Smi::kMinValue - 1.0),
-       BinaryOperationFeedback::kNumber},
+       BinaryOperationFeedback::kNumberOrOddball},
       {Token::Value::SUB, ast_factory.NewNumber(3.1415), ast_factory.NewSmi(3),
        isolate->factory()->NewHeapNumber(3.1415 - 3),
-       BinaryOperationFeedback::kNumber},
+       BinaryOperationFeedback::kNumberOrOddball},
       {Token::Value::SUB, ast_factory.NewNumber(3.1415),
        ast_factory.NewNumber(1.4142),
        isolate->factory()->NewHeapNumber(3.1415 - 1.4142),
-       BinaryOperationFeedback::kNumber},
+       BinaryOperationFeedback::kNumberOrOddball},
       {Token::Value::SUB, ast_factory.NewSmi(2),
        ast_factory.NewString(ast_factory.GetOneByteString("1")),
        Handle<Smi>(Smi::FromInt(1), isolate), BinaryOperationFeedback::kAny},
@@ -603,14 +603,14 @@ TEST(InterpreterBinaryOpTypeFeedback) {
        ast_factory.NewSmi(static_cast<uint32_t>(Smi::kMinValue)),
        ast_factory.NewSmi(2),
        isolate->factory()->NewHeapNumber(Smi::kMinValue * 2.0),
-       BinaryOperationFeedback::kNumber},
+       BinaryOperationFeedback::kNumberOrOddball},
       {Token::Value::MUL, ast_factory.NewNumber(3.1415), ast_factory.NewSmi(3),
        isolate->factory()->NewHeapNumber(3 * 3.1415),
-       BinaryOperationFeedback::kNumber},
+       BinaryOperationFeedback::kNumberOrOddball},
       {Token::Value::MUL, ast_factory.NewNumber(3.1415),
        ast_factory.NewNumber(1.4142),
        isolate->factory()->NewHeapNumber(3.1415 * 1.4142),
-       BinaryOperationFeedback::kNumber},
+       BinaryOperationFeedback::kNumberOrOddball},
       {Token::Value::MUL, ast_factory.NewSmi(2),
        ast_factory.NewString(ast_factory.GetOneByteString("1")),
        Handle<Smi>(Smi::FromInt(2), isolate), BinaryOperationFeedback::kAny},
@@ -623,11 +623,11 @@ TEST(InterpreterBinaryOpTypeFeedback) {
        BinaryOperationFeedback::kSignedSmallInputs},
       {Token::Value::DIV, ast_factory.NewNumber(3.1415), ast_factory.NewSmi(3),
        isolate->factory()->NewHeapNumber(3.1415 / 3),
-       BinaryOperationFeedback::kNumber},
+       BinaryOperationFeedback::kNumberOrOddball},
       {Token::Value::DIV, ast_factory.NewNumber(3.1415),
        ast_factory.NewNumber(-std::numeric_limits<double>::infinity()),
        isolate->factory()->NewHeapNumber(-0.0),
-       BinaryOperationFeedback::kNumber},
+       BinaryOperationFeedback::kNumberOrOddball},
       {Token::Value::DIV, ast_factory.NewSmi(2),
        ast_factory.NewString(ast_factory.GetOneByteString("1")),
        Handle<Smi>(Smi::FromInt(2), isolate), BinaryOperationFeedback::kAny},
@@ -637,14 +637,14 @@ TEST(InterpreterBinaryOpTypeFeedback) {
        BinaryOperationFeedback::kSignedSmall},
       {Token::Value::MOD, ast_factory.NewSmi(static_cast<uint32_t>(-4)),
        ast_factory.NewSmi(2), isolate->factory()->NewHeapNumber(-0.0),
-       BinaryOperationFeedback::kNumber},
+       BinaryOperationFeedback::kNumberOrOddball},
       {Token::Value::MOD, ast_factory.NewNumber(3.1415), ast_factory.NewSmi(3),
        isolate->factory()->NewHeapNumber(fmod(3.1415, 3.0)),
-       BinaryOperationFeedback::kNumber},
+       BinaryOperationFeedback::kNumberOrOddball},
       {Token::Value::MOD, ast_factory.NewNumber(-3.1415),
        ast_factory.NewNumber(-1.4142),
        isolate->factory()->NewHeapNumber(fmod(-3.1415, -1.4142)),
-       BinaryOperationFeedback::kNumber},
+       BinaryOperationFeedback::kNumberOrOddball},
       {Token::Value::MOD, ast_factory.NewSmi(3),
        ast_factory.NewString(ast_factory.GetOneByteString("-2")),
        Handle<Smi>(Smi::FromInt(1), isolate), BinaryOperationFeedback::kAny}};
@@ -701,10 +701,10 @@ TEST(InterpreterBinaryOpSmiTypeFeedback) {
        BinaryOperationFeedback::kSignedSmall},
       {Token::Value::ADD, ast_factory.NewSmi(2), Smi::kMaxValue,
        isolate->factory()->NewHeapNumber(Smi::kMaxValue + 2.0),
-       BinaryOperationFeedback::kNumber},
+       BinaryOperationFeedback::kNumberOrOddball},
       {Token::Value::ADD, ast_factory.NewNumber(3.1415), 2,
        isolate->factory()->NewHeapNumber(3.1415 + 2.0),
-       BinaryOperationFeedback::kNumber},
+       BinaryOperationFeedback::kNumberOrOddball},
       {Token::Value::ADD,
        ast_factory.NewString(ast_factory.GetOneByteString("2")), 2,
        isolate->factory()->NewStringFromAsciiChecked("22"),
@@ -716,10 +716,10 @@ TEST(InterpreterBinaryOpSmiTypeFeedback) {
       {Token::Value::SUB,
        ast_factory.NewSmi(static_cast<uint32_t>(Smi::kMinValue)), 1,
        isolate->factory()->NewHeapNumber(Smi::kMinValue - 1.0),
-       BinaryOperationFeedback::kNumber},
+       BinaryOperationFeedback::kNumberOrOddball},
       {Token::Value::SUB, ast_factory.NewNumber(3.1415), 2,
        isolate->factory()->NewHeapNumber(3.1415 - 2.0),
-       BinaryOperationFeedback::kNumber},
+       BinaryOperationFeedback::kNumberOrOddball},
       {Token::Value::SUB,
        ast_factory.NewString(ast_factory.GetOneByteString("2")), 2,
        Handle<Smi>(Smi::kZero, isolate), BinaryOperationFeedback::kAny},
@@ -729,7 +729,7 @@ TEST(InterpreterBinaryOpSmiTypeFeedback) {
        BinaryOperationFeedback::kSignedSmall},
       {Token::Value::BIT_OR, ast_factory.NewNumber(3.1415), 8,
        Handle<Smi>(Smi::FromInt(11), isolate),
-       BinaryOperationFeedback::kNumber},
+       BinaryOperationFeedback::kNumberOrOddball},
       {Token::Value::BIT_OR,
        ast_factory.NewString(ast_factory.GetOneByteString("2")), 1,
        Handle<Smi>(Smi::FromInt(3), isolate), BinaryOperationFeedback::kAny},
@@ -738,7 +738,8 @@ TEST(InterpreterBinaryOpSmiTypeFeedback) {
        Handle<Smi>(Smi::FromInt(1), isolate),
        BinaryOperationFeedback::kSignedSmall},
       {Token::Value::BIT_AND, ast_factory.NewNumber(3.1415), 2,
-       Handle<Smi>(Smi::FromInt(2), isolate), BinaryOperationFeedback::kNumber},
+       Handle<Smi>(Smi::FromInt(2), isolate),
+       BinaryOperationFeedback::kNumberOrOddball},
       {Token::Value::BIT_AND,
        ast_factory.NewString(ast_factory.GetOneByteString("2")), 1,
        Handle<Smi>(Smi::kZero, isolate), BinaryOperationFeedback::kAny},
@@ -748,7 +749,7 @@ TEST(InterpreterBinaryOpSmiTypeFeedback) {
        BinaryOperationFeedback::kSignedSmall},
       {Token::Value::SHL, ast_factory.NewNumber(3.1415), 2,
        Handle<Smi>(Smi::FromInt(12), isolate),
-       BinaryOperationFeedback::kNumber},
+       BinaryOperationFeedback::kNumberOrOddball},
       {Token::Value::SHL,
        ast_factory.NewString(ast_factory.GetOneByteString("2")), 1,
        Handle<Smi>(Smi::FromInt(4), isolate), BinaryOperationFeedback::kAny},
@@ -757,7 +758,8 @@ TEST(InterpreterBinaryOpSmiTypeFeedback) {
        Handle<Smi>(Smi::FromInt(1), isolate),
        BinaryOperationFeedback::kSignedSmall},
       {Token::Value::SAR, ast_factory.NewNumber(3.1415), 2,
-       Handle<Smi>(Smi::kZero, isolate), BinaryOperationFeedback::kNumber},
+       Handle<Smi>(Smi::kZero, isolate),
+       BinaryOperationFeedback::kNumberOrOddball},
       {Token::Value::SAR,
        ast_factory.NewString(ast_factory.GetOneByteString("2")), 1,
        Handle<Smi>(Smi::FromInt(1), isolate), BinaryOperationFeedback::kAny}};
@@ -854,12 +856,12 @@ TEST(InterpreterUnaryOpFeedback) {
 
     Object* feedback1 = callable.vector()->Get(slot1);
     CHECK(feedback1->IsSmi());
-    CHECK_EQ(BinaryOperationFeedback::kNumber,
+    CHECK_EQ(BinaryOperationFeedback::kNumberOrOddball,
              static_cast<Smi*>(feedback1)->value());
 
     Object* feedback2 = callable.vector()->Get(slot2);
     CHECK(feedback2->IsSmi());
-    CHECK_EQ(BinaryOperationFeedback::kNumber,
+    CHECK_EQ(BinaryOperationFeedback::kNumberOrOddball,
              static_cast<Smi*>(feedback2)->value());
 
     Object* feedback3 = callable.vector()->Get(slot3);
@@ -915,7 +917,7 @@ TEST(InterpreterBitwiseTypeFeedback) {
 
     Object* feedback1 = callable.vector()->Get(slot1);
     CHECK(feedback1->IsSmi());
-    CHECK_EQ(BinaryOperationFeedback::kNumber,
+    CHECK_EQ(BinaryOperationFeedback::kNumberOrOddball,
              static_cast<Smi*>(feedback1)->value());
 
     Object* feedback2 = callable.vector()->Get(slot2);