new classes: implement new.target passing to superclass constructor.

src/compiler.cc

Index: src/compiler.cc
diff --git a/src/compiler.cc b/src/compiler.cc
index e9cc11172664b92470eb0a9d0c530b4cbd3e303f..b1aa5b5fb1ff2f6c29f271364a0f824b8396b726 100644
--- a/src/compiler.cc
+++ b/src/compiler.cc
@@ -619,7 +619,11 @@ static void SetFunctionInfo(Handle<SharedFunctionInfo> function_info,
                             bool is_toplevel,
                             Handle<Script> script) {
   function_info->set_length(lit->parameter_count());
-  function_info->set_formal_parameter_count(lit->parameter_count());
+  if (FLAG_experimental_classes && IsSubclassConstructor(lit->kind())) {
+    function_info->set_formal_parameter_count(lit->parameter_count() + 1);

[arv (Not doing code reviews), 2015/02/09 17:44:39]

I'm worried about doing it at this high abstraction level. NewTarget is not a
formal parameter. Maybe it is better do it at a lower level abstraction in case
other code wants to use the formal parameter count for what it sounds like
(function length, rest params etc).

The SFI already has a field for the FunctionKind so deferring this choice seems
safer.

+  } else {
+    function_info->set_formal_parameter_count(lit->parameter_count());
+  }
   function_info->set_script(*script);
   function_info->set_function_token_position(lit->function_token_position());
   function_info->set_start_position(lit->start_position());