[Interpreter] Adds support for const/let variables to interpreter.

test/cctest/compiler/test-run-bytecode-graph-builder.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 <utility>
 
 #include "src/compiler/pipeline.h"
 #include "src/execution.h"
 #include "src/handles.h"
 #include "src/interpreter/bytecode-array-builder.h"
@@ skipping 2585 matching lines @@
           callable(factory->NewNumberFromInt(a)).ToHandleChecked();
       static const int results[] = {11, 12, 2};
       CHECK_EQ(Handle<Smi>::cast(return_val)->value(), results[a]);
     }
   }
 }
 
 TEST(BytecodeGraphBuilderDoExpressions) {
   bool old_flag = FLAG_harmony_do_expressions;
   FLAG_harmony_do_expressions = true;
-
   HandleAndZoneScope scope;
   Isolate* isolate = scope.main_isolate();
   Zone* zone = scope.main_zone();
   Factory* factory = isolate->factory();
   ExpectedSnippet<0> snippets[] = {
       {"var a = do {}; return a;", {factory->undefined_value()}},
       {"var a = do { var x = 100; }; return a;", {factory->undefined_value()}},
       {"var a = do { var x = 100; }; return a;", {factory->undefined_value()}},
       {"var a = do { var x = 100; x++; }; return a;",
        {handle(Smi::FromInt(100), isolate)}},
       {"var i = 0; for (; i < 5;) { i = do { if (i == 3) { break; }; i + 1; }};"
        "return i;",
        {handle(Smi::FromInt(3), isolate)}},
   };
 
   for (size_t i = 0; i < arraysize(snippets); i++) {
     ScopedVector<char> script(1024);
     SNPrintF(script, "function %s() { %s }\n%s();", kFunctionName,
              snippets[i].code_snippet, kFunctionName);
 
     BytecodeGraphTester tester(isolate, zone, script.start());
     auto callable = tester.GetCallable<>();
     Handle<Object> return_value = callable().ToHandleChecked();
     CHECK(return_value->SameValue(*snippets[i].return_value()));
   }
 
   FLAG_harmony_do_expressions = old_flag;
 }
 
+TEST(JumpOnHole) {

[rmcilroy, 2016/02/03 12:28:26]

Could you replicate the test-interpreter tests here rather than testing for just
JumpOnHole, I think that would be more useful.

[mythria, 2016/02/04 10:28:53]

Done. Now I have a 1500 line cl :)

+  HandleAndZoneScope scope;
+  Isolate* isolate = scope.main_isolate();
+  Zone* zone = scope.main_zone();
+  Factory* factory = isolate->factory();
+  // TODO(mythria): Add tests to check JumpIfHole. Initializing 'this' via super
+  // calls uses JumpIfHole.
+  ExpectedSnippet<0> snippets[] = {
+      {"let x = 1; x = 2; return x;", {factory->NewNumberFromInt(2)}},
+      {"const x = 3; return x;", {factory->NewNumberFromInt(3)}},
+  };
+
+  for (size_t i = 0; i < arraysize(snippets); i++) {
+    ScopedVector<char> script(1024);
+    SNPrintF(script, "function %s() { %s }\n%s();", kFunctionName,
+             snippets[i].code_snippet, kFunctionName);
+
+    BytecodeGraphTester tester(isolate, zone, script.start());
+    auto callable = tester.GetCallable<>();
+    Handle<Object> return_value = callable().ToHandleChecked();
+    CHECK(return_value->SameValue(*snippets[i].return_value()));
+  }
+}
+
 }  // namespace compiler
 }  // namespace internal
 }  // namespace v8