[Interpreter] Adds logical and, logical or and comma operators to interpreter

test/unittests/interpreter/bytecode-array-builder-unittest.cc

Index: test/unittests/interpreter/bytecode-array-builder-unittest.cc
diff --git a/test/unittests/interpreter/bytecode-array-builder-unittest.cc b/test/unittests/interpreter/bytecode-array-builder-unittest.cc
index 32acf9f15ca9a249adac1cd0b99926a36c733435..0d9057786a65410b3c3ebbb2baefc731b893098f 100644
--- a/test/unittests/interpreter/bytecode-array-builder-unittest.cc
+++ b/test/unittests/interpreter/bytecode-array-builder-unittest.cc
@@ -106,12 +106,14 @@ TEST_F(BytecodeArrayBuilderTest, AllBytecodesGenerated) {
   builder.Bind(&start);
   // Short jumps with Imm8 operands
   builder.Jump(&start).JumpIfTrue(&start).JumpIfFalse(&start);
+  builder.JumpIfToBooleanTrue(&start).JumpIfToBooleanFalse(&start);

[rmcilroy, 2015/10/13 15:43:24]

nit - you can avoid the builder. and just make this a followon - i.e.:

builder.Jump(&start).JumpIfTrue(&start).JumpIfFalse(&start)
    .JumpIfToBooleanTrue(&start).JumpIfToBooleanFalse(&start);

[mythria, 2015/10/14 13:33:42]

Done.

   // Insert dummy ops to force longer jumps
   for (int i = 0; i < 128; i++) {
     builder.LoadTrue();
   }
   // Longer jumps requiring Constant operand
   builder.Jump(&start).JumpIfTrue(&start).JumpIfFalse(&start);
+  builder.JumpIfToBooleanTrue(&start).JumpIfToBooleanFalse(&start);
   builder.Return();
 
   // Generate BytecodeArray.