[deoptimizer] Support materialization of closures.

[deoptimizer] Support materialization of closures.

This adds support to materialize JSFunction objects from deoptimization
information. By now we need to support this because TurboFan's escape
analysis can produce virtual (i.e. non-escaping) closures.

R=jarin@chromium.org

Committed: https://crrev.com/66d76797df41461f5b4751f3a6c9a8c71ced4316
Cr-Commit-Position: refs/heads/master@{#36400}

[Jarin, 2016-05-20 08:15:24]

lgtm.

https://codereview.chromium.org/1998143002/diff/1/src/deoptimizer.cc
File src/deoptimizer.cc (right):

https://codereview.chromium.org/1998143002/diff/1/src/deoptimizer.cc#newcode3718
src/deoptimizer.cc:3718: handle(isolate_->context()));
Don't you want to add a comment about the temporarily incorrect shared function
info and context?

[Michael Starzinger, 2016-05-20 08:32:39]

Thanks! Comments addressed. Landing.

https://codereview.chromium.org/1998143002/diff/1/src/deoptimizer.cc
File src/deoptimizer.cc (right):

https://codereview.chromium.org/1998143002/diff/1/src/deoptimizer.cc#newcode3718
src/deoptimizer.cc:3718: handle(isolate_->context()));
On 2016/05/20 08:15:24, Jarin wrote:
> Don't you want to add a comment about the temporarily incorrect shared
function
> info and context?

Done.

[Michael Starzinger, 2016-05-20 09:16:18]

The CQ bit was checked by mstarzinger@chromium.org

[Michael Starzinger, 2016-05-20 09:16:18]

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/1998143002/#ps20001
(title: "Addressed comment.")

[commit-bot: I haz the power, 2016-05-20 09:16:22]

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

[commit-bot: I haz the power, 2016-05-20 09:18:41]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-05-20 09:20:06]

Description was changed from

==========
[deoptimizer] Support materialization of closures.

This adds support to materialize JSFunction objects from deoptimization
information. By now we need to support this because TurboFan's escape
analysis can produce virtual (i.e. non-escaping) closures.

R=jarin@chromium.org
==========

to

==========
[deoptimizer] Support materialization of closures.

This adds support to materialize JSFunction objects from deoptimization
information. By now we need to support this because TurboFan's escape
analysis can produce virtual (i.e. non-escaping) closures.

R=jarin@chromium.org

Committed: https://crrev.com/66d76797df41461f5b4751f3a6c9a8c71ced4316
Cr-Commit-Position: refs/heads/master@{#36400}
==========

[commit-bot: I haz the power, 2016-05-20 09:20:07]

Patchset 2 (id:??) landed as
https://crrev.com/66d76797df41461f5b4751f3a6c9a8c71ced4316
Cr-Commit-Position: refs/heads/master@{#36400}