[es7] implement |do| expressions proposal

src/ast-literal-reindexer.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/ast-literal-reindexer.h"
 
 #include "src/ast.h"
 #include "src/scopes.h"
 
 namespace v8 {
@@ skipping 25 matching lines @@
 void AstLiteralReindexer::VisitBreakStatement(BreakStatement* node) {}
 
 
 void AstLiteralReindexer::VisitDebuggerStatement(DebuggerStatement* node) {}
 
 
 void AstLiteralReindexer::VisitNativeFunctionLiteral(
     NativeFunctionLiteral* node) {}
 
 
+void AstLiteralReindexer::VisitDoExpression(DoExpression* node) {
+  // TODO(caitp): literals in do expressions need re-indexing too.

[adamk, 2015/10/15 10:59:33]

Is this meant for a followup? It'd be nice to have a summary in the CL
description of what's known not to work in this patch.

[caitp (gmail), 2015/10/15 11:24:17]

I guess it's really the same as initializer literals, so maybe they don't need
reindexing

[adamk, 2015/10/15 12:01:25]

Seems like this should be testable one way or the other.

+}
+
+
 void AstLiteralReindexer::VisitLiteral(Literal* node) {}
 
 
 void AstLiteralReindexer::VisitRegExpLiteral(RegExpLiteral* node) {
   UpdateIndex(node);
 }
 
 
 void AstLiteralReindexer::VisitVariableProxy(VariableProxy* node) {}
 
@@ skipping 255 matching lines @@
 void AstLiteralReindexer::VisitFunctionLiteral(FunctionLiteral* node) {
   // We don't recurse into the declarations or body of the function literal:
 }
 
 
 void AstLiteralReindexer::Reindex(Expression* pattern) {
   pattern->Accept(this);
 }
 }  // namespace internal
 }  // namespace v8