[strong] Disallow implicit conversions for bitwise ops, shifts

test/mjsunit/strong/implicit-conversions.js

 // Copyright 2015 the V8 project authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // Flags: --strong-mode --allow-natives-syntax
 
 'use strict';
 
 // TODO(conradw): Implement other strong operators
-let strong_arith = [
-    "-",
-    "*",
-    "/",
-    "%"
-]
+let strong_binops = [

[arv (Not doing code reviews), 2015/04/24 15:39:42]

FYI. Our js style does not allow underscores in variable names. I know the V8
team has historically been opposed to follow the google style guide though...

[conradw, 2015/04/27 11:20:47]

Hah! I should at least try and be consistent, since I changed this elsewhere.

+  "-",
+  "*",
+  "/",
+  "%",
+  "|",
+  "&",
+  "^",
+  "<<",
+  ">>",
+  ">>>",
+];
 
-let nonnumber_values = [
-    "{}",
-    "'foo'",
-    "(function(){})",
-    "[]",
-    "'0'",
-    "'NaN'",
-    "(class Foo {})"
-]
+let strong_unops = [
+  "~"
+];
 
-let number_values = [
-    "0",
-    "(-0)",
-    "1",
-    "0.79",
-    "(-0.79)",
-    "4294967295",
-    "4294967296",
-    "(-4294967295)",
-    "(-4294967296)",
-    "9999999999999",
-    "(-9999999999999)",
-    "1.5e10",
-    "(-1.5e10)",
-    "0xFFF",
-    "(-0xFFF)",
-    "NaN",
-    "Infinity",
-    "(-Infinity)"
-]
+let nonNumberValues = [
+  "{}",
+  "'foo'",
+  "(function(){})",
+  "[]",
+  "'0'",
+  "'NaN'",
+  "(class Foo {})"
+];
+
+let numberValues = [
+  "0",
+  "(-0)",
+  "1",
+  "0.79",
+  "(-0.79)",
+  "4294967295",
+  "4294967296",
+  "(-4294967295)",
+  "(-4294967296)",
+  "9999999999999",
+  "(-9999999999999)",
+  "1.5e10",
+  "(-1.5e10)",
+  "0xFFF",
+  "(-0xFFF)",
+  "NaN",
+  "Infinity",
+  "(-Infinity)"
+];
 
 function sub_strong(x, y) {
   "use strong";
-  let v = x - y;
-  return v;
+  return x - y;
 }
 
 function mul_strong(x, y) {
   "use strong";
-  let v = x * y;
-  return v;
+  return x * y;
 }
 
 function div_strong(x, y) {
   "use strong";
-  let v = x / y;
-  return v;
+  return x / y;
 }
 
 function mod_strong(x, y) {
   "use strong";
-  let v = x % y;
-  return v;
+  return x % y;
 }
 
-let strong_funcs = [sub_strong, mul_strong, div_strong, mod_strong];
+function or_strong(x, y) {
+  "use strong";
+  return x | y;
+}
+
+function and_strong(x, y) {
+  "use strong";
+  return x & y;
+}
+
+function xor_strong(x, y) {
+  "use strong";
+  return x ^ y;
+}
+
+function shl_strong(x, y) {
+  "use strong";
+  return x << y;
+}
+
+function shr_strong(x, y) {
+  "use strong";
+  return x >> y;
+}
+
+function sar_strong(x, y) {
+  "use strong";
+  return x >>> y;
+}
+
+function not_strong(x, y) {

[arv (Not doing code reviews), 2015/04/24 15:39:42]

Not clear why you need this function. You are testing the unary ops later.

[conradw, 2015/04/27 11:20:47]

Yeah, it was a mistake to put this here.

+  "use strong";
+  let v1 = ~x;
+  let v2 = ~y;
+  return v1 | v2;
+}
+
+let strong_funcs = [sub_strong, mul_strong, div_strong, mod_strong, or_strong,
+                    and_strong, xor_strong, shl_strong, shr_strong, sar_strong,
+                    not_strong];
 
 function inline_sub_strong(x, y) {
   "use strong";
   let v = x - y;
   return v;
 }
 
 function inline_outer(x, y) {
   return inline_sub_strong(x, y);
 }
 
 function inline_sub(x, y) {
   let v = x - y;
   return v;
 }
 
 function inline_outer_strong(x, y) {
   "use strong";
   return inline_sub(x, y);
 }
 
-for (let op of strong_arith) {
-  for (let left of number_values) {
-    for (let right of number_values) {
+for (let op of strong_binops) {
+  for (let left of numberValues) {
+    for (let right of numberValues) {
       let expr = "(" + left + op + right + ")";
       assertEquals(eval(expr), eval("'use strong';" + expr));
       assertDoesNotThrow("'use strong'; " + expr + ";");
       assertDoesNotThrow("'use strong'; let v = " + expr + ";");
     }
   }
-  for (let left of number_values) {
-    for (let right of nonnumber_values) {
+  for (let left of numberValues) {
+    for (let right of nonNumberValues) {
       let expr = "(" + left + op + right + ")";
       assertDoesNotThrow("'use strict'; " + expr + ";");
       assertDoesNotThrow("'use strict'; let v = " + expr + ";");
       assertThrows("'use strong'; " + expr + ";", TypeError);
       assertThrows("'use strong'; let v = " + expr + ";", TypeError);
     }
   }
-  for (let left of nonnumber_values) {
-    for (let right of number_values.concat(nonnumber_values)) {
+  for (let left of nonNumberValues) {
+    for (let right of numberValues.concat(nonNumberValues)) {
       let expr = "(" + left + op + right + ")";
       assertDoesNotThrow("'use strict'; " + expr + ";");
       assertDoesNotThrow("'use strict'; let v = " + expr + ";");
       assertThrows("'use strong'; " + expr + ";", TypeError);
       assertThrows("'use strong'; let v = " + expr + ";", TypeError);
     }
   }
 }
 
+for (let op of strong_unops) {
+  for (let value of numberValues) {
+    let expr = "(" + op + value + ")";
+    assertEquals(eval(expr), eval("'use strong';" + expr));
+    assertDoesNotThrow("'use strong'; " + expr + ";");
+    assertDoesNotThrow("'use strong'; let v = " + expr + ";");
+  }
+  for (let value of nonNumberValues) {
+    let expr = "(" + op + value + ")";
+    assertDoesNotThrow("'use strict'; " + expr + ";");
+    assertDoesNotThrow("'use strict'; let v = " + expr + ";");
+    assertThrows("'use strong'; " + expr + ";", TypeError);
+    assertThrows("'use strong'; let v = " + expr + ";", TypeError);
+  }
+}
+
 for (let func of strong_funcs) {
   let a = func(4, 5);
   let b = func(4, 5);
   assertTrue(a === b);
   %OptimizeFunctionOnNextCall(func);
   let c = func(4, 5);
   assertOptimized(func);
   assertTrue(b === c);
   %DeoptimizeFunction(func);
   let d = func(4, 5);
   assertTrue(c === d);
   %DeoptimizeFunction(func);
   %ClearFunctionTypeFeedback(func);
 }
 
 for (let func of strong_funcs) {
   try {
     let a = func(2, 3);
     let b = func(2, 3);
     assertTrue(a === b);
     %OptimizeFunctionOnNextCall(func);
     let c = func(2, "foo");
     assertUnreachable();
-  } catch(e) {
-    assertTrue(e instanceof TypeError);
+  } catch (e) {
+    assertInstanceof(e, TypeError);
     assertUnoptimized(func);
     assertThrows(function(){func(2, "foo");}, TypeError);
     assertDoesNotThrow(function(){func(2, 3);});
   }
 }
 
 assertThrows(function(){inline_outer(1, {})}, TypeError);
 for (var i = 0; i < 100; i++) {
   inline_outer(1, 2);
 }
 assertThrows(function(){inline_outer(1, {})}, TypeError);
 
 assertDoesNotThrow(function(){inline_outer_strong(1, {})});
 for (var i = 0; i < 100; i++) {
   inline_outer_strong(1, 2);
 }
 assertDoesNotThrow(function(){inline_outer_strong(1, {})});