Introduce a BuiltinsConstructStub that sets up new.target and does a [[call]] per ES6 9.3.2

src/runtime/runtime-classes.cc

 // Copyright 2014 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/runtime/runtime-utils.h"
 
 #include <stdlib.h>
 #include <limits>
 
 #include "src/arguments.h"
@@ skipping 124 matching lines @@
   Map::SetPrototype(map, prototype_parent);
   map->SetConstructor(*constructor);
   Handle<JSObject> prototype = isolate->factory()->NewJSObjectFromMap(map);
 
   Handle<String> name_string = name->IsString()
                                    ? Handle<String>::cast(name)
                                    : isolate->factory()->empty_string();
   constructor->shared()->set_name(*name_string);
 
   if (!super_class->IsTheHole()) {
-    Handle<Code> stub(isolate->builtins()->JSConstructStubForDerived());
+    Handle<Code> stub(isolate->builtins()->JSBuiltinsConstructStub());

[Igor Sheludko, 2015/11/18 09:16:07]

Please add a comment about why do we use JSBuiltinsConstructStub for derived
classes to avoid confusion.

[Toon Verwaest, 2015/11/18 13:24:57]

Done.

     constructor->shared()->set_construct_stub(*stub);
   }
 
   JSFunction::SetPrototype(constructor, prototype);
   PropertyAttributes attribs =
       static_cast<PropertyAttributes>(DONT_ENUM | DONT_DELETE | READ_ONLY);
   RETURN_ON_EXCEPTION(isolate,
                       JSObject::SetOwnPropertyIgnoreAttributes(
                           constructor, isolate->factory()->prototype_string(),
                           prototype, attribs),
@@ skipping 341 matching lines @@
   Handle<Object> result;
   ASSIGN_RETURN_FAILURE_ON_EXCEPTION(
       isolate, result, Execution::New(isolate, super_constructor, new_target,
                                       argument_count, arguments.get()));
 
   return *result;
 }
 
 }  // namespace internal
 }  // namespace v8