[Interpreter] Add support for for count operations.

test/cctest/interpreter/test-interpreter.cc

 // 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.
 
 #include "src/v8.h"
 
 #include "src/execution.h"
 #include "src/handles.h"
 #include "src/interpreter/bytecode-array-builder.h"
 #include "src/interpreter/interpreter.h"
@@ skipping 1981 matching lines @@
 
   // TODO(rmcilroy): modify tests when we have real try finally support.
   std::string source(InterpreterTester::SourceForBody(
       "var a = 1; try { a = a + 1; } finally { a = a + 2; }; return a;"));
   InterpreterTester tester(handles.main_isolate(), source.c_str());
   auto callable = tester.GetCallable<>();
 
   Handle<Object> return_val = callable().ToHandleChecked();
   CHECK_EQ(Smi::cast(*return_val), Smi::FromInt(4));
 }
+
+
+TEST(InterpreterCountOperators) {
+  HandleAndZoneScope handles;
+  i::Isolate* isolate = handles.main_isolate();
+  i::Factory* factory = isolate->factory();
+
+  std::pair<const char*, Handle<Object>> count_ops[16] = {
+      std::make_pair("var a = 1; return ++a;",
+                     Handle<Object>(Smi::FromInt(2), isolate)),

[Toon Verwaest, 2015/10/20 13:42:28]

you could just write handle(Smi::FromInt(2), isolate)

[rmcilroy, 2015/10/22 12:47:56]

Good call, done this throughout, thanks!

+      std::make_pair("var a = 1; return a++;",
+                     Handle<Object>(Smi::FromInt(1), isolate)),
+      std::make_pair("var a = 5; return --a;",
+                     Handle<Object>(Smi::FromInt(4), isolate)),
+      std::make_pair("var a = 5; return a--;",
+                     Handle<Object>(Smi::FromInt(5), isolate)),
+      std::make_pair("var a = 5.2; return --a;",
+                     factory->NewHeapNumber(4.2)),
+      std::make_pair("var a = 'string'; return ++a;",
+                     factory->nan_value()),
+      std::make_pair("var a = 'string'; return a--;",
+                     factory->nan_value()),
+      std::make_pair("var a = true; return ++a;",
+                     Handle<Object>(Smi::FromInt(2), isolate)),
+      std::make_pair("var a = false; return a--;",
+                     Handle<Object>(Smi::FromInt(0), isolate)),
+      std::make_pair("var a = { val: 11 }; return ++a.val;",
+                     Handle<Object>(Smi::FromInt(12), isolate)),
+      std::make_pair("var a = { val: 11 }; return a.val--;",
+                     Handle<Object>(Smi::FromInt(11), isolate)),
+      std::make_pair("var a = { val: 11 }; return ++a.val;",
+                     Handle<Object>(Smi::FromInt(12), isolate)),
+      std::make_pair("var name = 'val'; var a = { val: 22 }; return --a[name];",
+                     Handle<Object>(Smi::FromInt(21), isolate)),
+      std::make_pair("var name = 'val'; var a = { val: 22 }; return a[name]++;",
+                     Handle<Object>(Smi::FromInt(22), isolate)),
+      std::make_pair("var a = 1; (function() { a = 2 })(); return ++a;",
+                     Handle<Object>(Smi::FromInt(3), isolate)),
+      std::make_pair("var a = 1; (function() { a = 2 })(); return a--;",
+                     Handle<Object>(Smi::FromInt(2), isolate)),
+  };
+
+  for (size_t i = 0; i < arraysize(count_ops); i++) {
+    std::string source(InterpreterTester::SourceForBody(count_ops[i].first));
+    InterpreterTester tester(handles.main_isolate(), source.c_str());
+    auto callable = tester.GetCallable<>();
+
+    Handle<i::Object> return_value = callable().ToHandleChecked();
+    CHECK(return_value->SameValue(*count_ops[i].second));
+  }
+}
+
+
+TEST(InterpreterGlobalCountOperators) {
+  HandleAndZoneScope handles;
+  i::Isolate* isolate = handles.main_isolate();
+
+  std::pair<const char*, Handle<Object>> count_ops[6] = {
+      std::make_pair("var global = 100;function f(){ return ++global; }",
+                     Handle<Object>(Smi::FromInt(101), isolate)),
+      std::make_pair("var global = 100; function f(){ return --global; }",
+                     Handle<Object>(Smi::FromInt(99), isolate)),
+      std::make_pair("var global = 100; function f(){ return global++; }",
+                     Handle<Object>(Smi::FromInt(100), isolate)),
+      std::make_pair("unallocated = 200; function f(){ return ++unallocated; }",
+                     Handle<Object>(Smi::FromInt(201), isolate)),
+      std::make_pair("unallocated = 200; function f(){ return --unallocated; }",
+                     Handle<Object>(Smi::FromInt(199), isolate)),
+      std::make_pair("unallocated = 200; function f(){ return unallocated++; }",
+                     Handle<Object>(Smi::FromInt(200), isolate)),
+  };
+
+  for (size_t i = 0; i < arraysize(count_ops); i++) {
+    InterpreterTester tester(handles.main_isolate(), count_ops[i].first);
+    auto callable = tester.GetCallable<>();
+
+    Handle<i::Object> return_value = callable().ToHandleChecked();
+    CHECK(return_value->SameValue(*count_ops[i].second));
+  }
+}