[api] Stay in C++ when constructing an API-function

[api] Stay in C++ when constructing an API-function

This CL applies the same optimization already present for calling API-function.
Execution::New and Execution::Call now both check whether the target is an
API-function and avoid calling out into the construct/call stub.

BUG=chromium:630217
Committed: https://crrev.com/b920d5f3ff33e8202f3218521c8335723d384e03
Cr-Commit-Position: refs/heads/master@{#38433}

[Camillo Bruni, 2016-08-03 12:48:40]

cbruni@chromium.org changed reviewers:
+ verwaest@chromium.org

[Camillo Bruni, 2016-08-03 12:48:41]

PTAL

[Toon Verwaest, 2016-08-04 12:31:05]

lgtm

https://codereview.chromium.org/2203353002/diff/1/src/execution.cc
File src/execution.cc (right):

https://codereview.chromium.org/2203353002/diff/1/src/execution.cc#newcode63
src/execution.cc:63: if (target->IsJSFunction() &&
Move this below the stackoverflow check below.

[Camillo Bruni, 2016-08-04 13:01:51]

The CQ bit was checked by cbruni@chromium.org

[Camillo Bruni, 2016-08-04 13:01:52]

The patchset sent to the CQ was uploaded after l-g-t-m from
verwaest@chromium.org
Link to the patchset: https://codereview.chromium.org/2203353002/#ps20001
(title: "moving stack overflow check")

[commit-bot: I haz the power, 2016-08-04 13:01:58]

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, 2016-08-04 13:15:06]

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

[commit-bot: I haz the power, 2016-08-04 13:15:07]

Try jobs failed on following builders:
  v8_linux64_gyp_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_gyp_rel_ng/build...)
  v8_linux64_gyp_rel_ng_triggered on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_gyp_rel_ng_trigg...)

[Camillo Bruni, 2016-08-04 13:15:55]

The CQ bit was unchecked by cbruni@chromium.org

[Camillo Bruni, 2016-08-08 10:01:47]

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

[commit-bot: I haz the power, 2016-08-08 10:01:54]

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, 2016-08-08 10:32:53]

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

[commit-bot: I haz the power, 2016-08-08 10:32:54]

Dry run: This issue passed the CQ dry run.

[Camillo Bruni, 2016-08-08 11:13:50]

The CQ bit was checked by cbruni@chromium.org

[Camillo Bruni, 2016-08-08 11:13:50]

The patchset sent to the CQ was uploaded after l-g-t-m from
verwaest@chromium.org
Link to the patchset: https://codereview.chromium.org/2203353002/#ps60001
(title: "set the receiver to the_hole for construct calls")

[commit-bot: I haz the power, 2016-08-08 11:13:56]

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, 2016-08-08 11:16:20]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-08-08 11:17:00]

Description was changed from

==========
[api] Stay in C++ when constructing an API-function

This CL applies the same optimization already present for calling API-function.
Execution::New and Execution::Call now both check whether the target is an
API-function and avoid calling out into the construct/call stub.

BUG=chromium:630217
==========

to

==========
[api] Stay in C++ when constructing an API-function

This CL applies the same optimization already present for calling API-function.
Execution::New and Execution::Call now both check whether the target is an
API-function and avoid calling out into the construct/call stub.

BUG=chromium:630217

Committed: https://crrev.com/b920d5f3ff33e8202f3218521c8335723d384e03
Cr-Commit-Position: refs/heads/master@{#38433}
==========

[commit-bot: I haz the power, 2016-08-08 11:17:01]

Patchset 4 (id:??) landed as
https://crrev.com/b920d5f3ff33e8202f3218521c8335723d384e03
Cr-Commit-Position: refs/heads/master@{#38433}