[runtime] Make target checking for %Call and %_Call consistent.

[runtime] Make target checking for %Call and %_Call consistent.

The %_Call intrinsic (if supported by the compiler) is lowered directly
to the Call builtin and thus throws a TypeError if the target is not
callable. The %Call runtime function also eventually calls into the Call
builtin, but had an early abort if the target is not a JSReceiver, which
is unnecessary and leads to various test failures for Ignition.

R=mvstanton@chromium.org

Committed: https://crrev.com/458764621b2b667ea349d0a8fd8361cc99d7bf51
Cr-Commit-Position: refs/heads/master@{#34316}

[Benedikt Meurer, 2016-02-26 06:40:14]

Hey Michael,

Here's an easy consistency fix for %Call vs %_Call, which was blowing up in
Ignition.
Please take a look.

Thanks,
Benedikt

Ross: FYI

[mvstanton, 2016-02-26 06:42:51]

lgtm

[Benedikt Meurer, 2016-02-26 06:43:02]

The CQ bit was checked by bmeurer@chromium.org

[commit-bot: I haz the power, 2016-02-26 06:43:09]

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

[commit-bot: I haz the power, 2016-02-26 06:59:12]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-02-26 07:00:41]

Description was changed from

==========
[runtime] Make target checking for %Call and %_Call consistent.

The %_Call intrinsic (if supported by the compiler) is lowered directly
to the Call builtin and thus throws a TypeError if the target is not
callable. The %Call runtime function also eventually calls into the Call
builtin, but had an early abort if the target is not a JSReceiver, which
is unnecessary and leads to various test failures for Ignition.

R=mvstanton@chromium.org
==========

to

==========
[runtime] Make target checking for %Call and %_Call consistent.

The %_Call intrinsic (if supported by the compiler) is lowered directly
to the Call builtin and thus throws a TypeError if the target is not
callable. The %Call runtime function also eventually calls into the Call
builtin, but had an early abort if the target is not a JSReceiver, which
is unnecessary and leads to various test failures for Ignition.

R=mvstanton@chromium.org

Committed: https://crrev.com/458764621b2b667ea349d0a8fd8361cc99d7bf51
Cr-Commit-Position: refs/heads/master@{#34316}
==========

[commit-bot: I haz the power, 2016-02-26 07:00:42]

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

[rmcilroy, 2016-02-26 08:40:11]

rmcilroy@chromium.org changed reviewers:
+ rmcilroy@chromium.org

[rmcilroy, 2016-02-26 08:40:11]

Awesome thanks Benedikt. +Mythri who was looking at these test failures
yesterday.