[async-await] Remove call indirection in Create and Reject

src/js/promise.js

 // 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.
 
 (function(global, utils, extrasUtils) {
 
 "use strict";
 
 %CheckIsBootstrapping();
 
@@ skipping 54 matching lines @@
 // Define exported functions.
 
 // For bootstrapper.
 
 // This is used by utils and v8-extras.
 function PromiseCreate() {
   return %promise_internal_constructor(UNDEFINED);
 }
 
 // Only used by async-await.js
 function RejectPromise(promise, reason, debugEvent) {

[gsathya, 2016/12/22 16:19:01]

Can this be removed?

[Dan Ehrenberg, 2016/12/22 16:22:32]

Yeah, seems like we could remove promise_internal_reject if we just update its
one user PromiseRejectClosure to call the runtime function directly. I'm OK with
that in a separate patch or the same patch.

   %PromiseReject(promise, reason, debugEvent);
 }
 
 // Export to bindings
 function DoRejectPromise(promise, reason) {
   %PromiseReject(promise, reason, true);
 }
 
 // The resultCapability.promise is only ever fulfilled internally,
 // so we don't need the closures to protect against accidentally
@@ skipping 269 matching lines @@
 // This allows extras to create promises quickly without building extra
 // resolve/reject closures, and allows them to later resolve and reject any
 // promise without having to hold on to those closures forever.
 utils.InstallFunctions(extrasUtils, 0, [
   "createPromise", PromiseCreate,
   "rejectPromise", DoRejectPromise,
   "markPromiseAsHandled", MarkPromiseAsHandled
 ]);
 
 utils.Export(function(to) {
-  to.PromiseCreate = PromiseCreate;
-
   to.CreateInternalPromiseCapability = CreateInternalPromiseCapability;
-  to.RejectPromise = RejectPromise;
 });
 
 })