[turbofan] add ZonePool to correctly track compiler phase memory usage

[turbofan] add ZonePool to correctly track compiler phase memory usage

R=bmeurer@chromium.org

BUG=

Committed: https://code.google.com/p/v8/source/detail?r=24771

[dcarney, 2014-10-21 10:44:41]

Patchset #1 (id:1) has been deleted

[dcarney, 2014-10-21 10:46:23]

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

[dcarney, 2014-10-21 10:47:25]

ptal

I added tracking for Scheduler but there are lots of zones still missing in
--turbo-stats

[Benedikt Meurer, 2014-10-21 10:55:33]

Some unit tests would be nice.

https://codereview.chromium.org/665893006/diff/20001/src/compiler/zone-pool.cc
File src/compiler/zone-pool.cc (right):

https://codereview.chromium.org/665893006/diff/20001/src/compiler/zone-pool.c...
src/compiler/zone-pool.cc:84: Zone* ZonePool::GetEmptyZone() {
This should not be called Get*.

[dcarney, 2014-10-21 12:24:26]

On 2014/10/21 10:55:33, Benedikt Meurer wrote:
> Some unit tests would be nice.

Done.

> https://codereview.chromium.org/665893006/diff/20001/src/compiler/zone-pool.cc
> File src/compiler/zone-pool.cc (right):
> 
>
https://codereview.chromium.org/665893006/diff/20001/src/compiler/zone-pool.c...
> src/compiler/zone-pool.cc:84: Zone* ZonePool::GetEmptyZone() {
> This should not be called Get*.

What do you prefer?

[Benedikt Meurer, 2014-10-21 12:25:19]

NewEmptyZone() I guess.

[dcarney, 2014-10-21 12:29:08]

On 2014/10/21 12:25:19, Benedikt Meurer wrote:
> NewEmptyZone() I guess.

done

[Benedikt Meurer, 2014-10-21 12:29:29]

lgtm

[dcarney, 2014-10-21 12:39:03]

Committed patchset #3 (id:60001) manually as 24771 (presubmit successful).