[strong] Disallow implicit conversions for binary arithmetic operations

src/code-stubs.h

Index: src/code-stubs.h
diff --git a/src/code-stubs.h b/src/code-stubs.h
index 424026f40b24b27390278541d7b35f341017548d..a6bd45f876b984d655ae5bc82a9f5ccfe25c4798 100644
--- a/src/code-stubs.h
+++ b/src/code-stubs.h
@@ -1248,9 +1248,9 @@ class CallApiGetterStub : public PlatformCodeStub {
 
 class BinaryOpICStub : public HydrogenCodeStub {
  public:
-  BinaryOpICStub(Isolate* isolate, Token::Value op)
+  BinaryOpICStub(Isolate* isolate, Token::Value op, LanguageMode language_mode)
       : HydrogenCodeStub(isolate, UNINITIALIZED) {
-    BinaryOpICState state(isolate, op);
+    BinaryOpICState state(isolate, op, language_mode);

[rossberg, 2015/04/23 13:29:37]

I think it's right for the BinoryOpICStub constructor to take the language mode.
However, BinaryOpICState should only take a bool, since otherwise we'd start
distinguishing sloppy vs strict in the state and hence stub key, which implies
that we'll cache them separately.

[conradw, 2015/04/23 14:51:54]

Done.

     set_sub_minor_key(state.GetExtraICState());
   }
 
@@ -1331,8 +1331,9 @@ class BinaryOpICWithAllocationSiteStub final : public PlatformCodeStub {
 
 class BinaryOpWithAllocationSiteStub final : public BinaryOpICStub {
  public:
-  BinaryOpWithAllocationSiteStub(Isolate* isolate, Token::Value op)
-      : BinaryOpICStub(isolate, op) {}
+  BinaryOpWithAllocationSiteStub(Isolate* isolate, Token::Value op,
+                                 LanguageMode language_mode)
+      : BinaryOpICStub(isolate, op, language_mode) {}
 
   BinaryOpWithAllocationSiteStub(Isolate* isolate, const BinaryOpICState& state)
       : BinaryOpICStub(isolate, state) {}