[es6] support rest parameters in arrow functions

src/full-codegen.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/v8.h"
 
 #include "src/ast.h"
 #include "src/ast-numbering.h"
 #include "src/code-factory.h"
 #include "src/codegen.h"
@@ skipping 270 matching lines @@
   }
 }
 
 
 void BreakableStatementChecker::VisitCompareOperation(CompareOperation* expr) {
   Visit(expr->left());
   Visit(expr->right());
 }
 
 
+void BreakableStatementChecker::VisitSpreadOperation(SpreadOperation* expr) {
+  UNREACHABLE();

[arv (Not doing code reviews), 2015/03/10 13:44:02]

This does not seem right. Shouldn't this be:

Visit(expr->expression());

... or maybe I am thinking to far ahead when we have array and call spread.

[caitp (gmail), 2015/03/10 14:47:51]

eventually yeah, but for now it's not really needed since these nodes are
removed from the graph

+}
+
+
 void BreakableStatementChecker::VisitThisFunction(ThisFunction* expr) {
 }
 
 
 void BreakableStatementChecker::VisitSuperReference(SuperReference* expr) {}
 
 
 #define __ ACCESS_MASM(masm())
 
 bool FullCodeGenerator::MakeCode(CompilationInfo* info) {
@@ skipping 640 matching lines @@
       return VisitComma(expr);
     case Token::OR:
     case Token::AND:
       return VisitLogicalExpression(expr);
     default:
       return VisitArithmeticExpression(expr);
   }
 }
 
 
+void FullCodeGenerator::VisitSpreadOperation(SpreadOperation* node) {
+  UNREACHABLE();
+}
+
+
 void FullCodeGenerator::VisitInDuplicateContext(Expression* expr) {
   if (context()->IsEffect()) {
     VisitForEffect(expr);
   } else if (context()->IsAccumulatorValue()) {
     VisitForAccumulatorValue(expr);
   } else if (context()->IsStackValue()) {
     VisitForStackValue(expr);
   } else if (context()->IsTest()) {
     const TestContext* test = TestContext::cast(context());
     VisitForControl(expr, test->true_label(), test->false_label(),
@@ skipping 857 matching lines @@
   }
   codegen_->PrepareForBailoutForId(exit_id_, NO_REGISTERS);
   codegen_->scope_ = saved_scope_;
 }
 
 
 #undef __
 
 
 } }  // namespace v8::internal