[promises] Move promise constructor to TFS

src/builtins/builtins.h

Index: src/builtins/builtins.h
diff --git a/src/builtins/builtins.h b/src/builtins/builtins.h
index de061528be0a768c58fbf9fd6b89dabaed7a3caf..5aa87cdb31aafa6db3d63222346b14ed16366658 100644
--- a/src/builtins/builtins.h
+++ b/src/builtins/builtins.h
@@ -563,6 +563,9 @@ namespace internal {
   TFS(ForInFilter, BUILTIN, kNoExtraICState, ForInFilter)                     \
                                                                               \
   /* Promise */                                                               \
+  TFJ(PromiseConstructor, 1)                                                  \
+  TFJ(PromiseInternalConstructor, 0)                                          \
+  TFJ(IsPromise, 1)                                                           \

[caitp, 2016/11/29 13:33:00]

Maybe for the internal functions, we should skip the arguments adaptor frame by
using SharedFunctionInfo::kDontAdaptArgumentsSentinel as the arg count, and just
add a CSA_SLOW_ASSERT() to ensure it's the right count in debug builds

[gsathya, 2016/12/01 20:03:53]

AFAIK, this sets up the number of stack parameters which needs to be 1 here. The
adaptor frame creation is determined by
SharedFunctionInfo::internal_formal_parameter_count (which is set to
kDontAdaptArgumentsSentinel in the bootstrapper for this function).

[caitp, 2016/12/01 20:12:37]

well, what's going on with StringFromCharCode then? I don't know, wasn't CC'd on
that bug when that change landed. I'd have to take a closer look to see what
this really does.

   CPP(CreateResolvingFunctions)                                               \
   CPP(PromiseResolveClosure)                                                  \
   CPP(PromiseRejectClosure)                                                   \