[es6] early error when Identifier is an escaped reserved word

src/parser.cc

 // Copyright 2012 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/parser.h"
 
 #include "src/api.h"
 #include "src/ast.h"
 #include "src/ast-literal-reindexer.h"
 #include "src/bailout-reason.h"
@@ skipping 1407 matching lines @@
         return ParseVariableStatement(kStatementListItem, NULL, ok);
       }
       break;
     case Token::VAR:
       return ParseVariableStatement(kStatementListItem, NULL, ok);
     case Token::LET:
       if (IsNextLetKeyword()) {
         return ParseVariableStatement(kStatementListItem, NULL, ok);
       }
       break;
+    case Token::IDENTIFIER:
+      CheckNextEscapedKeyword(ok);

[adamk, 2015/11/03 23:00:57]

You can CHECK_OK, and then please break.

[caitp (gmail), 2015/11/04 04:49:24]

CHECK_OK means moving the macro definition here, but sgtm. Done.

+      if (!*ok) return nullptr;
     default:
       break;
   }
   return ParseStatement(NULL, ok);
 }
 
 
 Statement* Parser::ParseModuleItem(bool* ok) {
   // (Ecma 262 6th Edition, 15.2):
   // ModuleItem :
@@ skipping 4943 matching lines @@
 
 Expression* Parser::SpreadCallNew(Expression* function,
                                   ZoneList<v8::internal::Expression*>* args,
                                   int pos) {
   args->InsertAt(0, function, zone());
 
   return factory()->NewCallRuntime(Context::REFLECT_CONSTRUCT_INDEX, args, pos);
 }
 }  // namespace internal
 }  // namespace v8