[builtins] Refactor Invoke to deal with any kind of callable.

[builtin] Refactor Invoke to deal with any kind of callable.

Now both Execution::Call and Execution::New can deal with any
kind of target and will raise a proper exception if the target is not
callable (which is not yet spec compliant for New, as we would
have to check IsConstructor instead, which we don't have yet).

Now we no longer need to do any of these weird call/construct
delegate gymnastics in C++, and we finally have a single true
bottleneck for Call/Construct abstract operations in the code
base, with only a few special handlings left in the compilers to
optimize the JSFunction case.

R=jarin@chromium.org
BUG=v8:4430, v8:4413
LOG=n

Committed: https://crrev.com/634d1d86d8e98584097c6cb548a68ecf705c44f7
Cr-Commit-Position: refs/heads/master@{#30874}

[Benedikt Meurer, 2015-09-22 12:53:14]

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

[commit-bot: I haz the power, 2015-09-22 12:53:20]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1360793002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1360793002/20001

[Benedikt Meurer, 2015-09-22 12:53:47]

Patchset #1 (id:1) has been deleted

[commit-bot: I haz the power, 2015-09-22 12:54:24]

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

[commit-bot: I haz the power, 2015-09-22 12:54:25]

Dry run: Try jobs failed on following builders:
  v8_linux_arm_rel on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_arm_rel/builds/8256)

[Benedikt Meurer, 2015-09-22 17:56:28]

Patchset #1 (id:20001) has been deleted

[Benedikt Meurer, 2015-09-22 17:56:42]

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

[commit-bot: I haz the power, 2015-09-22 17:57:01]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1360793002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1360793002/40001

[commit-bot: I haz the power, 2015-09-22 17:59:21]

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

[commit-bot: I haz the power, 2015-09-22 17:59:22]

Dry run: Try jobs failed on following builders:
  v8_linux_mips64el_compile_rel on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_mips64el_compile_r...)

[Benedikt Meurer, 2015-09-23 02:39:56]

Patchset #1 (id:40001) has been deleted

[Benedikt Meurer, 2015-09-23 02:40:09]

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

[commit-bot: I haz the power, 2015-09-23 02:40:24]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1360793002/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1360793002/60001

[commit-bot: I haz the power, 2015-09-23 02:44:58]

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

[commit-bot: I haz the power, 2015-09-23 02:44:58]

Dry run: Try jobs failed on following builders:
  v8_linux_arm64_rel on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_arm64_rel/builds/9836)

[Benedikt Meurer, 2015-09-23 04:14:56]

Patchset #1 (id:60001) has been deleted

[Benedikt Meurer, 2015-09-23 04:18:02]

bmeurer@chromium.org changed reviewers:
+ jarin@chromium.org

[Benedikt Meurer, 2015-09-23 04:19:11]

Hey Jaro,

Finally got rid of the horrible C++ magic around Invoke!
Please take a look.

Thanks,
Benedikt

[Benedikt Meurer, 2015-09-23 04:48:35]

Patchset #1 (id:80001) has been deleted

[Benedikt Meurer, 2015-09-23 04:48:40]

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

[commit-bot: I haz the power, 2015-09-23 04:49:01]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1360793002/100001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1360793002/100001

[commit-bot: I haz the power, 2015-09-23 05:10:41]

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

[commit-bot: I haz the power, 2015-09-23 05:10:42]

Dry run: This issue passed the CQ dry run.

[Jarin, 2015-09-23 05:40:37]

lgtm

https://codereview.chromium.org/1360793002/diff/100001/src/arm/builtins-arm.cc
File src/arm/builtins-arm.cc (right):

https://codereview.chromium.org/1360793002/diff/100001/src/arm/builtins-arm.c...
src/arm/builtins-arm.cc:1670: // 2. Construct of something that else, which
might have a [[Construct]]
This comment does not make a lot of sense to me.

Maybe you meant:

Construct something else that might...

[Benedikt Meurer, 2015-09-23 05:41:54]

https://codereview.chromium.org/1360793002/diff/100001/src/arm/builtins-arm.cc
File src/arm/builtins-arm.cc (right):

https://codereview.chromium.org/1360793002/diff/100001/src/arm/builtins-arm.c...
src/arm/builtins-arm.cc:1670: // 2. Construct of something that else, which
might have a [[Construct]]
I will fix that in the CL that introduces IsConstructor.

[Benedikt Meurer, 2015-09-23 05:42:01]

The CQ bit was checked by bmeurer@chromium.org

[commit-bot: I haz the power, 2015-09-23 05:42:22]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1360793002/100001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1360793002/100001

[commit-bot: I haz the power, 2015-09-23 05:44:05]

Committed patchset #1 (id:100001)

[commit-bot: I haz the power, 2015-09-23 05:44:20]

Patchset 1 (id:??) landed as
https://crrev.com/634d1d86d8e98584097c6cb548a68ecf705c44f7
Cr-Commit-Position: refs/heads/master@{#30874}