Change the CompilerDispatcherJob to take a SharedFunctionInfo

Change the CompilerDispatcherJob to take a SharedFunctionInfo

This means we can no longer take the closure's context to parse, but
need to rely on the outer scope info.

Since it's not possible to get that, however, for lazy functions, we
introduce a new field to SharedFunctionInfo that stores the outer scope
info whenever available.

BUG=v8:5215
R=marja@chromium.org,verwaest@chromium.org,jgruber@chromium.org

Committed: https://crrev.com/0f0912dd23911eec6d1c5e6d3ebba1ece0ff3361
Cr-Commit-Position: refs/heads/master@{#39548}

[jochen (gone - plz use gerrit), 2016-09-20 08:10:03]

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

[commit-bot: I haz the power, 2016-09-20 08:10:16]

Dry run: 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, 2016-09-20 08:52:18]

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

[commit-bot: I haz the power, 2016-09-20 08:52:19]

Dry run: Try jobs failed on following builders:
  v8_linux_dbg_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_dbg_ng/builds/13035)
  v8_linux_dbg_ng_triggered on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_dbg_ng_triggered/b...)

[jochen (gone - plz use gerrit), 2016-09-20 09:25:09]

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

[commit-bot: I haz the power, 2016-09-20 09:25:14]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[jochen (gone - plz use gerrit), 2016-09-20 09:25:24]

jochen@chromium.org changed reviewers:
+ hpayer@chromium.org

[jochen (gone - plz use gerrit), 2016-09-20 09:25:24]

+hpayer for heap/

[jochen (gone - plz use gerrit), 2016-09-20 09:34:13]

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

[commit-bot: I haz the power, 2016-09-20 09:34:21]

Dry run: 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, 2016-09-20 10:08:56]

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

[commit-bot: I haz the power, 2016-09-20 10:08:57]

Dry run: This issue passed the CQ dry run.

[Toon Verwaest, 2016-09-20 11:36:15]

lgtm

https://codereview.chromium.org/2358503002/diff/40001/src/compiler-dispatcher...
File src/compiler-dispatcher/compiler-dispatcher-job.cc (right):

https://codereview.chromium.org/2358503002/diff/40001/src/compiler-dispatcher...
src/compiler-dispatcher/compiler-dispatcher-job.cc:73:
parse_info_->set_calls_eval(shared_->scope_info()->CallsEval());
Just drop this line. We'll (re)compute it while parsing.

[jochen (gone - plz use gerrit), 2016-09-20 11:37:52]

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

[commit-bot: I haz the power, 2016-09-20 11:37:59]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Hannes Payer (out of office), 2016-09-20 11:39:23]

heap lgtm

[jochen (gone - plz use gerrit), 2016-09-20 11:40:38]

The CQ bit was unchecked by jochen@chromium.org

[jochen (gone - plz use gerrit), 2016-09-20 11:40:41]

The CQ bit was checked by jochen@chromium.org

[jochen (gone - plz use gerrit), 2016-09-20 11:40:41]

The patchset sent to the CQ was uploaded after l-g-t-m from
verwaest@chromium.org
Link to the patchset: https://codereview.chromium.org/2358503002/#ps60001
(title: "updates")

[commit-bot: I haz the power, 2016-09-20 11:40:46]

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, 2016-09-20 12:07:56]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-09-20 12:08:53]

Description was changed from

==========
Change the CompilerDispatcherJob to take a SharedFunctionInfo

This means we can no longer take the closure's context to parse, but
need to rely on the outer scope info.

Since it's not possible to get that, however, for lazy functions, we
introduce a new field to SharedFunctionInfo that stores the outer scope
info whenever available.

BUG=v8:5215
R=marja@chromium.org,verwaest@chromium.org,jgruber@chromium.org
==========

to

==========
Change the CompilerDispatcherJob to take a SharedFunctionInfo

This means we can no longer take the closure's context to parse, but
need to rely on the outer scope info.

Since it's not possible to get that, however, for lazy functions, we
introduce a new field to SharedFunctionInfo that stores the outer scope
info whenever available.

BUG=v8:5215
R=marja@chromium.org,verwaest@chromium.org,jgruber@chromium.org

Committed: https://crrev.com/0f0912dd23911eec6d1c5e6d3ebba1ece0ff3361
Cr-Commit-Position: refs/heads/master@{#39548}
==========

[commit-bot: I haz the power, 2016-09-20 12:08:54]

Patchset 4 (id:??) landed as
https://crrev.com/0f0912dd23911eec6d1c5e6d3ebba1ece0ff3361
Cr-Commit-Position: refs/heads/master@{#39548}