[async-await] Move PromiseCreate and PromiseRelease to TF

[async-await] Move PromiseCreate and PromiseRelease to TF

BUG=v8:5639
Review-Url: https://codereview.chromium.org/2638073002
Cr-Commit-Position: refs/heads/master@{#42508}
Committed: https://chromium.googlesource.com/v8/v8/+/d37c407a81763bf41f82db915697a65fcf20d5ca

[jgruber, 2017-01-17 16:19:26]

The CQ bit was checked by jgruber@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-01-17 16:19:36]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-01-17 16:21:06]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-01-17 16:21:07]

Dry run: Try jobs failed on following builders:
  v8_android_arm_compile_rel on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_android_arm_compile_rel/...)
  v8_linux64_avx2_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_avx2_rel_ng/buil...)
  v8_linux64_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_rel_ng/builds/19331)
  v8_linux_arm_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_arm_rel_ng/builds/...)
  v8_linux_gcc_compile_rel on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_gcc_compile_rel/bu...)
  v8_linux_mips64el_compile_rel on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_mips64el_compile_r...)
  v8_linux_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_rel_ng/builds/19300)
  v8_presubmit on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_presubmit/builds/32582)
  v8_win_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_win_rel_ng/builds/21040)

[jgruber, 2017-01-17 16:23:41]

jgruber@chromium.org changed reviewers:
+ gsathya@chromium.org, littledan@chromium.org

[jgruber, 2017-01-17 16:23:42]

Will fix the patch errors tomorrow.

[jgruber, 2017-01-18 12:53:37]

The CQ bit was checked by jgruber@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-01-18 12:53:47]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[jgruber, 2017-01-18 13:14:17]

And done, PTAL.

[commit-bot: I haz the power, 2017-01-18 13:22:54]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-01-18 13:22:55]

Dry run: This issue passed the CQ dry run.

[gsathya, 2017-01-18 18:58:42]

lgtm

https://codereview.chromium.org/2638073002/diff/20001/src/builtins/builtins-a...
File src/builtins/builtins-async-await.cc (right):

https://codereview.chromium.org/2638073002/diff/20001/src/builtins/builtins-a...
src/builtins/builtins-async-await.cc:28: Node* const promise =
AllocateAndInitJSPromise(context, parent);
You don't need the parent here, default is undefined

[jgruber, 2017-01-19 08:43:23]

The CQ bit was checked by jgruber@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-01-19 08:43:29]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-01-19 08:44:59]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-01-19 08:45:00]

Dry run: Try jobs failed on following builders:
  v8_android_arm_compile_rel on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_android_arm_compile_rel/...)
  v8_linux64_avx2_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_avx2_rel_ng/buil...)
  v8_linux64_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_rel_ng/builds/19503)
  v8_linux_arm64_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_arm64_rel_ng/build...)
  v8_linux_arm_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_arm_rel_ng/builds/...)
  v8_linux_dbg_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_dbg_ng/builds/19571)
  v8_presubmit on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_presubmit/builds/32810)
  v8_win64_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_win64_rel_ng/builds/21122)

[jgruber, 2017-01-19 08:55:03]

The CQ bit was checked by jgruber@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-01-19 08:55:09]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[jgruber, 2017-01-19 08:58:11]

+yang for src/js, PTAL

I renamed builtins-async-await.cc to builtins-async-function.cc since that seems
to be the current naming convention (and async-await makes less sense with
caitp's upcoming async-generator and async-iterator files).

gsathya@ suggested moving debug logic to desugaring in an offline discussion.
Makes sense to me but I'll save it for a follow-up commit.

https://codereview.chromium.org/2638073002/diff/20001/src/builtins/builtins-a...
File src/builtins/builtins-async-await.cc (right):

https://codereview.chromium.org/2638073002/diff/20001/src/builtins/builtins-a...
src/builtins/builtins-async-await.cc:28: Node* const promise =
AllocateAndInitJSPromise(context, parent);
On 2017/01/18 18:58:42, gsathya wrote:
> You don't need the parent here, default is undefined

Done.

[commit-bot: I haz the power, 2017-01-19 09:29:52]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-01-19 09:29:53]

Dry run: This issue passed the CQ dry run.

[jgruber, 2017-01-19 12:19:40]

jgruber@chromium.org changed reviewers:
+ yangguo@chromium.org

[Yang, 2017-01-19 15:12:33]

lgtm

[jgruber, 2017-01-19 15:13:30]

The CQ bit was checked by jgruber@chromium.org

[jgruber, 2017-01-19 15:13:31]

The patchset sent to the CQ was uploaded after l-g-t-m from gsathya@chromium.org
Link to the patchset: https://codereview.chromium.org/2638073002/#ps60001
(title: "Move to builtins-async-function.cc and address comment")

[commit-bot: I haz the power, 2017-01-19 15:13:42]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-01-19 15:15:38]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1484838810777050,
"parent_rev": "d22934bd9fe3e2aea4bb62772204850e757c0636", "commit_rev":
"d37c407a81763bf41f82db915697a65fcf20d5ca"}

[commit-bot: I haz the power, 2017-01-19 15:15:42]

Description was changed from

==========
[async-await] Move PromiseCreate and PromiseRelease to TF

BUG=v8:5639
==========

to

==========
[async-await] Move PromiseCreate and PromiseRelease to TF

BUG=v8:5639

Review-Url: https://codereview.chromium.org/2638073002
Cr-Commit-Position: refs/heads/master@{#42508}
Committed:
https://chromium.googlesource.com/v8/v8/+/d37c407a81763bf41f82db915697a65fcf2...
==========

[commit-bot: I haz the power, 2017-01-19 15:15:43]

Committed patchset #4 (id:60001) as
https://chromium.googlesource.com/v8/v8/+/d37c407a81763bf41f82db915697a65fcf2...