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

src/js/promise.js

Index: src/js/promise.js
diff --git a/src/js/promise.js b/src/js/promise.js
index d84d70df4ec8eadcb700c260117759ebeac2195f..f0c9bb3b520ca1bda70fff1ad1f3baae3933a00d 100644
--- a/src/js/promise.js
+++ b/src/js/promise.js
@@ -61,11 +61,15 @@ function CreateResolvingFunctions(promise) {
 
 
 var GlobalPromise = function Promise(resolver) {
-  if (resolver === promiseRawSymbol) return;
-  if (!%_IsConstructCall()) throw MakeTypeError(kNotAPromise, this);
+  if (resolver === promiseRawSymbol) {
+    return %NewObject(GlobalPromise, new.target);
+  }
+  if (IS_UNDEFINED(new.target)) throw MakeTypeError(kNotAPromise, this);

[Igor Sheludko, 2015/11/19 09:30:53]

Perhaps this should be done even before checking resolver.

[Toon Verwaest, 2015/11/19 09:39:53]

%NewObject internally will check that new.target is a JSReceiver (and
constructable) in debug mode. That should be fine...

   if (!IS_CALLABLE(resolver))
     throw MakeTypeError(kResolverNotAFunction, resolver);
-  var promise = PromiseInit(this);
+
+  var promise = PromiseInit(%NewObject(GlobalPromise, new.target));
+
   try {
     %DebugPushPromise(promise, Promise, resolver);
     var callbacks = CreateResolvingFunctions(promise);
@@ -75,6 +79,8 @@ var GlobalPromise = function Promise(resolver) {
   } finally {
     %DebugPopPromise();
   }
+
+  return promise;
 }
 
 // Core functionality.