Don't count nested function scopes towards MaxNestedContextChainlength

Don't count nested function scopes towards MaxNestedContextChainlength

This is allocating registers in the function for all inner contexts that can be active in that function, so that nested blocks always have O(1) access to all outer contexts. However, currently it's always walking into nested functions, overallocating the number of registers, causing additional register pressure.

BUG=v8:5484
Committed: https://crrev.com/35fc3d58a1f63edf979474aae7a91fc64f5867f6
Cr-Commit-Position: refs/heads/master@{#40214}

[Toon Verwaest, 2016-10-12 10:22:17]

verwaest@chromium.org changed reviewers:
+ marja@chromium.org

[Toon Verwaest, 2016-10-12 10:22:18]

ptal

[jochen (gone - plz use gerrit), 2016-10-12 10:41:08]

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

[jochen (gone - plz use gerrit), 2016-10-12 10:41:09]

that would explain why we couldn't get it to crash

[marja, 2016-10-12 11:50:07]

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

[marja, 2016-10-12 11:50:08]

lgtm, this is convincing, but I added bmeurer@ as a reviewer since he knows how
this works :)

[Toon Verwaest, 2016-10-12 11:52:10]

The CQ bit was checked by verwaest@chromium.org

[commit-bot: I haz the power, 2016-10-12 11:52:16]

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-12 12:14:32]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-10-12 12:15:10]

Description was changed from

==========
Don't count nested function scopes towards MaxNestedContextChainlength

This is allocating registers in the function for all inner contexts that can be
active in that function, so that nested blocks always have O(1) access to all
outer contexts. However, currently it's always walking into nested functions,
overallocating the number of registers, causing additional register pressure.

BUG=v8:5484
==========

to

==========
Don't count nested function scopes towards MaxNestedContextChainlength

This is allocating registers in the function for all inner contexts that can be
active in that function, so that nested blocks always have O(1) access to all
outer contexts. However, currently it's always walking into nested functions,
overallocating the number of registers, causing additional register pressure.

BUG=v8:5484

Committed: https://crrev.com/35fc3d58a1f63edf979474aae7a91fc64f5867f6
Cr-Commit-Position: refs/heads/master@{#40214}
==========

[commit-bot: I haz the power, 2016-10-12 12:15:11]

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