Turn ZoneList inner_scopes into a linked list

Turn ZoneList inner_scopes into a linked list

Replace the zonelist with a link from a scope to any of its inner scopes, and a link to any sibling scope.
This makes scopes that track inner scopes use roughly the same amount of space as previously scopes without inner scopes would use for the empty zonelist (pointer to the memory + length field, which, granted could be slightly smaller on 64bit).

BUG=v8:5209
Committed: https://crrev.com/f04fd635b6914336d5bbf2f4928139c239945387
Cr-Commit-Position: refs/heads/master@{#37941}

[Toon Verwaest, 2016-07-20 12:05:15]

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

[Toon Verwaest, 2016-07-20 12:05:15]

ptal

[Yang, 2016-07-20 12:08:46]

On 2016/07/20 12:05:15, Toon Verwaest wrote:
> ptal

src/debug lgtm.

[marja, 2016-07-21 07:03:06]

lgtm \o/

This is cool stuff, the efficient version is also cleaner / more elegant, imo!

[Toon Verwaest, 2016-07-21 08:51:30]

The CQ bit was checked by verwaest@chromium.org

[Toon Verwaest, 2016-07-21 08:51:30]

The patchset sent to the CQ was uploaded after l-g-t-m from yangguo@chromium.org
Link to the patchset: https://codereview.chromium.org/2162143005/#ps40001
(title: "Even more properly reparent ... :)")

[commit-bot: I haz the power, 2016-07-21 08:51:39]

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-07-21 08:52:44]

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

[commit-bot: I haz the power, 2016-07-21 08:52:45]

Try jobs failed on following builders:
  v8_linux64_avx2_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_avx2_rel_ng/buil...)
  v8_linux64_gyp_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_gyp_rel_ng/build...)
  v8_linux_dbg_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_dbg_ng/builds/9441)
  v8_linux_mipsel_compile_rel on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_mipsel_compile_rel...)
  v8_mac_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_mac_rel_ng/builds/5424)
  v8_presubmit on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_presubmit/builds/19929)

[Toon Verwaest, 2016-07-21 09:00:55]

The CQ bit was checked by verwaest@chromium.org

[Toon Verwaest, 2016-07-21 09:00:55]

The patchset sent to the CQ was uploaded after l-g-t-m from
yangguo@chromium.org, marja@chromium.org
Link to the patchset: https://codereview.chromium.org/2162143005/#ps60001
(title: "rebase")

[commit-bot: I haz the power, 2016-07-21 09:01:04]

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-07-21 11:01:41]

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

[commit-bot: I haz the power, 2016-07-21 11:01:42]

Try jobs failed on following builders:
  v8_linux_dbg_ng on master.tryserver.v8 (JOB_TIMED_OUT, no build URL)

[Toon Verwaest, 2016-07-21 11:07:12]

The CQ bit was checked by verwaest@chromium.org

[commit-bot: I haz the power, 2016-07-21 11:07:20]

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-07-21 11:23:40]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-07-21 11:26:01]

Description was changed from

==========
Turn ZoneList inner_scopes into a linked list

Replace the zonelist with a link from a scope to any of its inner scopes, and a
link to any sibling scope.
This makes scopes that track inner scopes use roughly the same amount of space
as previously scopes without inner scopes would use for the empty zonelist
(pointer to the memory + length field, which, granted could be slightly smaller
on 64bit).

BUG=v8:5209
==========

to

==========
Turn ZoneList inner_scopes into a linked list

Replace the zonelist with a link from a scope to any of its inner scopes, and a
link to any sibling scope.
This makes scopes that track inner scopes use roughly the same amount of space
as previously scopes without inner scopes would use for the empty zonelist
(pointer to the memory + length field, which, granted could be slightly smaller
on 64bit).

BUG=v8:5209

Committed: https://crrev.com/f04fd635b6914336d5bbf2f4928139c239945387
Cr-Commit-Position: refs/heads/master@{#37941}
==========

[commit-bot: I haz the power, 2016-07-21 11:26:02]

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