[deoptimizer] Teach the Deoptimizer about bound functions.

[deoptimizer] Teach the Deoptimizer about bound functions.

The inlining of Function.prototype.bind can lead to escape analyzed
bound functions, which weren't handled by the Deoptimizer previously.

BUG=chromium:729573
R=jarin@chromium.org,cbruni@chromium.org

Review-Url: https://codereview.chromium.org/2931483003
Cr-Commit-Position: refs/heads/master@{#45751}
Committed: https://chromium.googlesource.com/v8/v8/+/337bb36e5251efd0db641a9f200d19cb101229fa

[Benedikt Meurer, 2017-06-07 05:36:53]

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

[commit-bot: I haz the power, 2017-06-07 05:37:02]

Dry run: CQ is trying da patch.

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

[Jarin, 2017-06-07 05:42:47]

lgtm

https://codereview.chromium.org/2931483003/diff/1/test/mjsunit/regress/regres...
File test/mjsunit/regress/regress-crbug-729573.js (right):

https://codereview.chromium.org/2931483003/diff/1/test/mjsunit/regress/regres...
test/mjsunit/regress/regress-crbug-729573.js:11: if (!a) return a;
Please, write a test that calls the materialized function, so that it makes sure
that the object is materialized correctly.

[Benedikt Meurer, 2017-06-07 05:55:44]

The CQ bit was checked by bmeurer@chromium.org

[Benedikt Meurer, 2017-06-07 05:55:44]

The patchset sent to the CQ was uploaded after l-g-t-m from jarin@chromium.org
Link to the patchset: https://codereview.chromium.org/2931483003/#ps20001
(title: "Add moar test covfefe")

[Benedikt Meurer, 2017-06-07 05:55:50]

https://codereview.chromium.org/2931483003/diff/1/test/mjsunit/regress/regres...
File test/mjsunit/regress/regress-crbug-729573.js (right):

https://codereview.chromium.org/2931483003/diff/1/test/mjsunit/regress/regres...
test/mjsunit/regress/regress-crbug-729573.js:11: if (!a) return a;
On 2017/06/07 05:42:46, Jarin wrote:
> Please, write a test that calls the materialized function, so that it makes
sure
> that the object is materialized correctly.

Done.

[commit-bot: I haz the power, 2017-06-07 05:55:55]

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-06-07 06:25:27]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1496814944238150,
"parent_rev": "645d4a463f41ac3e814d48e340689354335fc502", "commit_rev":
"337bb36e5251efd0db641a9f200d19cb101229fa"}

[commit-bot: I haz the power, 2017-06-07 06:25:33]

Description was changed from

==========
[deoptimizer] Teach the Deoptimizer about bound functions.

The inlining of Function.prototype.bind can lead to escape analyzed
bound functions, which weren't handled by the Deoptimizer previously.

BUG=chromium:729573
R=jarin@chromium.org,cbruni@chromium.org
==========

to

==========
[deoptimizer] Teach the Deoptimizer about bound functions.

The inlining of Function.prototype.bind can lead to escape analyzed
bound functions, which weren't handled by the Deoptimizer previously.

BUG=chromium:729573
R=jarin@chromium.org,cbruni@chromium.org

Review-Url: https://codereview.chromium.org/2931483003
Cr-Commit-Position: refs/heads/master@{#45751}
Committed:
https://chromium.googlesource.com/v8/v8/+/337bb36e5251efd0db641a9f200d19cb101...
==========

[commit-bot: I haz the power, 2017-06-07 06:25:35]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/v8/v8/+/337bb36e5251efd0db641a9f200d19cb101...