[strong] Implement strong mode semantics for the count operation.

src/compiler/ast-graph-builder.cc

Index: src/compiler/ast-graph-builder.cc
diff --git a/src/compiler/ast-graph-builder.cc b/src/compiler/ast-graph-builder.cc
index 89373a37bca255dfad94d1bb0d2eae11685375d9..0759f13c63e7b9b0778d93c88500c17a6be4aadc 100644
--- a/src/compiler/ast-graph-builder.cc
+++ b/src/compiler/ast-graph-builder.cc
@@ -2661,9 +2661,11 @@ void AstGraphBuilder::VisitCountOperation(CountOperation* expr) {
   }
 
   // Convert old value into a number.
-  old_value = NewNode(javascript()->ToNumber(), old_value);
-  PrepareFrameState(old_value, expr->ToNumberId(),
-                    OutputFrameStateCombine::Push());
+  if (!is_strong(language_mode())) {
+    old_value = NewNode(javascript()->ToNumber(), old_value);
+    PrepareFrameState(old_value, expr->ToNumberId(),
+                      OutputFrameStateCombine::Push());
+  }
 
   // TODO(titzer): combine this framestate with the above?
   FrameStateBeforeAndAfter store_states(this, assign_type == KEYED_PROPERTY
@@ -2679,9 +2681,10 @@ void AstGraphBuilder::VisitCountOperation(CountOperation* expr) {
     FrameStateBeforeAndAfter states(this, BailoutId::None());
     value =
         BuildBinaryOp(old_value, jsgraph()->OneConstant(), expr->binary_op());
-    // This should never deoptimize because we have converted to number
-    // before.
-    states.AddToNode(value, BailoutId::None(),
+    // This should never deoptimize in sloppy mode because we have converted to

[rossberg, 2015/06/30 12:53:52]

s/sloppy/weak/ ?

[conradw, 2015/06/30 14:01:10]

reworded

+    // number before.
+    states.AddToNode(value, is_strong(language_mode()) ? expr->ToNumberId()
+                                                       : BailoutId::None(),
                      OutputFrameStateCombine::Ignore());
   }