[parser] Lazy compilation no longer needs context chain.

[parser] Lazy compilation no longer needs context chain.

Now that the scope chain is deserialized directly from the chain of
{ScopeInfo} objects, it is no longer needed to provide a context. This
makes the {AllowsLazyCompilationWithoutContext} predicate coincide with
the more general {AllowsLazyCompilation}. Remove the former.

R=jochen@chromium.org

Committed: https://crrev.com/8f5d1c1d4c652f020352af8505baa29804a3ef3d
Cr-Commit-Position: refs/heads/master@{#40042}

[Michael Starzinger, 2016-10-06 12:35:32]

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

[commit-bot: I haz the power, 2016-10-06 12:35:41]

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-10-06 13:05:50]

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

[commit-bot: I haz the power, 2016-10-06 13:05:51]

Dry run: This issue passed the CQ dry run.

[jochen (gone - plz use gerrit), 2016-10-06 13:58:48]

https://codereview.chromium.org/2399853002/diff/1/src/objects.cc
File src/objects.cc (right):

https://codereview.chromium.org/2399853002/diff/1/src/objects.cc#newcode13754
src/objects.cc:13754: shared_info->set_allows_lazy_compilation_without_context(
why not remove allows_lazy_compilation_without_context from SFI?

[Michael Starzinger, 2016-10-06 14:03:28]

https://codereview.chromium.org/2399853002/diff/1/src/objects.cc
File src/objects.cc (right):

https://codereview.chromium.org/2399853002/diff/1/src/objects.cc#newcode13754
src/objects.cc:13754: shared_info->set_allows_lazy_compilation_without_context(
On 2016/10/06 13:58:48, jochen wrote:
> why not remove allows_lazy_compilation_without_context from SFI?

I am working on a follow-up CL doing that as we speak. There is one remaining
case that disables lazy compilation without context for top-level eval code. It
is triggered here:

https://cs.chromium.org/chromium/src/v8/src/compiler.cc?q=set_allows_lazy_com...

I can merge that work into this CL if you prefer or leave it as a separate CL.
Your call.

[jochen (gone - plz use gerrit), 2016-10-06 14:04:52]

lgtm

[Michael Starzinger, 2016-10-06 14:15:45]

The CQ bit was checked by mstarzinger@chromium.org

[commit-bot: I haz the power, 2016-10-06 14:15: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, 2016-10-06 14:17:55]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-10-06 14:18:20]

Description was changed from

==========
[parser] Lazy compilation no longer needs context chain.

Now that the scope chain is deserialized directly from the chain of
{ScopeInfo} objects, it is no longer needed to provide a context. This
makes the {AllowsLazyCompilationWithoutContext} predicate coincide with
the more general {AllowsLazyCompilation}. Remove the former.

R=jochen@chromium.org
==========

to

==========
[parser] Lazy compilation no longer needs context chain.

Now that the scope chain is deserialized directly from the chain of
{ScopeInfo} objects, it is no longer needed to provide a context. This
makes the {AllowsLazyCompilationWithoutContext} predicate coincide with
the more general {AllowsLazyCompilation}. Remove the former.

R=jochen@chromium.org

Committed: https://crrev.com/8f5d1c1d4c652f020352af8505baa29804a3ef3d
Cr-Commit-Position: refs/heads/master@{#40042}
==========

[commit-bot: I haz the power, 2016-10-06 14:18:21]

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