[crankshaft] Check if the function is callable before generating a tail call via Call builtin.

[crankshaft] Check if the function is callable before generating a tail call via Call builtin.

This is necessary to ensure that "Called non callable" exception will get a proper message and stack trace even for calls at tail position.

BUG=chromium:595615, v8:4698
LOG=N

Committed: https://crrev.com/e6dca379b633682e64542c86ed68941ccca221f0
Cr-Commit-Position: refs/heads/master@{#34962}

[Igor Sheludko, 2016-03-21 11:51:47]

Description was changed from

==========
[crankshaft] Check if the function is callable before generating a call to Call
builtin.

This is necessary to ensure that "Called non callable" exception will get a
proper message and stack trace even for calls at tail position.

BUG=chromium:595615, v8:4698
LOG=N
==========

to

==========
[crankshaft] Check if the function is callable before generating a tail call via
Call builtin.

This is necessary to ensure that "Called non callable" exception will get a
proper message and stack trace even for calls at tail position.

BUG=chromium:595615, v8:4698
LOG=N
==========

[Igor Sheludko, 2016-03-21 11:53:09]

Patchset #2 (id:20001) has been deleted

[Igor Sheludko, 2016-03-21 11:53:15]

Patchset #1 (id:1) has been deleted

[Igor Sheludko, 2016-03-21 11:53:49]

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

[commit-bot: I haz the power, 2016-03-21 11:53:58]

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

[Igor Sheludko, 2016-03-21 12:20:55]

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

[Igor Sheludko, 2016-03-21 12:20:55]

PTAL

[commit-bot: I haz the power, 2016-03-21 14:03:12]

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

[commit-bot: I haz the power, 2016-03-21 14:03:13]

Dry run: This issue passed the CQ dry run.

[Benedikt Meurer, 2016-03-21 18:00:36]

lgtm

[Igor Sheludko, 2016-03-21 18:54:52]

The CQ bit was checked by ishell@chromium.org

[commit-bot: I haz the power, 2016-03-21 18:54:58]

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

[commit-bot: I haz the power, 2016-03-21 19:22:45]

Description was changed from

==========
[crankshaft] Check if the function is callable before generating a tail call via
Call builtin.

This is necessary to ensure that "Called non callable" exception will get a
proper message and stack trace even for calls at tail position.

BUG=chromium:595615, v8:4698
LOG=N
==========

to

==========
[crankshaft] Check if the function is callable before generating a tail call via
Call builtin.

This is necessary to ensure that "Called non callable" exception will get a
proper message and stack trace even for calls at tail position.

BUG=chromium:595615, v8:4698
LOG=N
==========

[commit-bot: I haz the power, 2016-03-21 19:22:49]

Committed patchset #1 (id:40001)

[commit-bot: I haz the power, 2016-03-21 19:24:36]

Description was changed from

==========
[crankshaft] Check if the function is callable before generating a tail call via
Call builtin.

This is necessary to ensure that "Called non callable" exception will get a
proper message and stack trace even for calls at tail position.

BUG=chromium:595615, v8:4698
LOG=N
==========

to

==========
[crankshaft] Check if the function is callable before generating a tail call via
Call builtin.

This is necessary to ensure that "Called non callable" exception will get a
proper message and stack trace even for calls at tail position.

BUG=chromium:595615, v8:4698
LOG=N

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

[commit-bot: I haz the power, 2016-03-21 19:24:36]

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