Call binary op stub instead of runtime in count operations.

src/ia32/codegen-ia32.cc

 // Copyright 2010 the V8 project authors. All rights reserved.
 // Redistribution and use in source and binary forms, with or without
 // modification, are permitted provided that the following conditions are
 // met:
 //
 //     * Redistributions of source code must retain the above copyright
 //       notice, this list of conditions and the following disclaimer.
 //     * Redistributions in binary form must reproduce the above
 //       copyright notice, this list of conditions and the following
 //       disclaimer in the documentation and/or other materials provided
@@ skipping 7016 matching lines @@
 };
 
 
 void DeferredPrefixCountOperation::Generate() {
   // Undo the optimistic smi operation.
   if (is_increment_) {
     __ sub(Operand(dst_), Immediate(Smi::FromInt(1)));
   } else {
     __ add(Operand(dst_), Immediate(Smi::FromInt(1)));
   }
-  __ push(dst_);
-  if (!input_type_.IsNumber()) {
+  Register left;
+  if (input_type_.IsNumber()) {
+    left = dst_;
+  } else {
+    __ push(dst_);
     __ InvokeBuiltin(Builtins::TO_NUMBER, CALL_FUNCTION);
-    __ push(eax);
+    left = eax;
   }
-  __ push(Immediate(Smi::FromInt(1)));
-  if (is_increment_) {
-    __ CallRuntime(Runtime::kNumberAdd, 2);
-  } else {
-    __ CallRuntime(Runtime::kNumberSub, 2);
-  }
+
+  GenericBinaryOpStub stub(is_increment_ ? Token::ADD : Token::SUB,
+                           NO_OVERWRITE,
+                           NO_GENERIC_BINARY_FLAGS,
+                           TypeInfo::Number());
+  stub.GenerateCall(masm_, left, Smi::FromInt(1));
+
   if (!dst_.is(eax)) __ mov(dst_, eax);
 }
 
 
 // The value in dst was optimistically incremented or decremented.  The
 // result overflowed or was not smi tagged.  Undo the operation and call
 // into the runtime to convert the argument to a number.  Update the
 // original value in old.  Call the specialized add or subtract stub.
 // The result is left in dst.
 class DeferredPostfixCountOperation: public DeferredCode {
@@ skipping 19 matching lines @@
 };
 
 
 void DeferredPostfixCountOperation::Generate() {
   // Undo the optimistic smi operation.
   if (is_increment_) {
     __ sub(Operand(dst_), Immediate(Smi::FromInt(1)));
   } else {
     __ add(Operand(dst_), Immediate(Smi::FromInt(1)));
   }
+  Register left;
   if (input_type_.IsNumber()) {
     __ push(dst_);  // Save the input to use as the old value.
-    __ push(dst_);
+    left = dst_;
   } else {
     __ push(dst_);
     __ InvokeBuiltin(Builtins::TO_NUMBER, CALL_FUNCTION);
     __ push(eax);  // Save the result of ToNumber to use as the old value.
-    __ push(eax);
+    left = eax;
   }
 
-  // Call the runtime for the addition or subtraction.
-  __ push(Immediate(Smi::FromInt(1)));
-  if (is_increment_) {
-    __ CallRuntime(Runtime::kNumberAdd, 2);
-  } else {
-    __ CallRuntime(Runtime::kNumberSub, 2);
-  }
+  GenericBinaryOpStub stub(is_increment_ ? Token::ADD : Token::SUB,
+                           NO_OVERWRITE,
+                           NO_GENERIC_BINARY_FLAGS,
+                           TypeInfo::Number());
+  stub.GenerateCall(masm_, left, Smi::FromInt(1));
+
   if (!dst_.is(eax)) __ mov(dst_, eax);
   __ pop(old_);
 }
 
 
 void CodeGenerator::VisitCountOperation(CountOperation* node) {
   ASSERT(!in_safe_int32_mode());
   Comment cmnt(masm_, "[ CountOperation");
 
   bool is_postfix = node->is_postfix();
@@ skipping 5486 matching lines @@
 
   // Call the runtime; it returns -1 (less), 0 (equal), or 1 (greater)
   // tagged as a small integer.
   __ bind(&runtime);
   __ TailCallRuntime(Runtime::kStringCompare, 2, 1);
 }
 
 #undef __
 
 } }  // namespace v8::internal