[turbofan] delete graph and schedule before register allocation.

[turbofan] delete graph and schedule before register allocation.

R=mstarzinger@chromium.org, titzer@chromium.org

BUG=

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

[dcarney, 2014-10-20 13:20:53]

Patchset #1 (id:1) has been deleted

[dcarney, 2014-10-20 13:21:53]

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

[dcarney, 2014-10-20 13:22:00]

ptal

[titzer, 2014-10-20 14:01:43]

titzer@chromium.org changed reviewers:
+ mstarzinger@chromium.org

[titzer, 2014-10-20 14:01:43]

+mstarzinger

[dcarney, 2014-10-21 07:14:33]

Patchset #1 (id:20001) has been deleted

[dcarney, 2014-10-21 15:29:34]

Patchset #1 (id:40001) has been deleted

[dcarney, 2014-10-21 15:29:40]

Patchset #1 (id:60001) has been deleted

[Michael Starzinger, 2014-10-23 12:18:36]

LGTM, just a coupl'a nits.

https://codereview.chromium.org/662513005/diff/100001/src/compiler/pipeline.cc
File src/compiler/pipeline.cc (right):

https://codereview.chromium.org/662513005/diff/100001/src/compiler/pipeline.c...
src/compiler/pipeline.cc:117: typer_ = NULL;
If the schedule lives in the graph-zone, then we should also set the schedule to
NULL here.

https://codereview.chromium.org/662513005/diff/100001/src/compiler/pipeline.c...
src/compiler/pipeline.cc:137: // graph_zone_ contains everything up to
instruction_zone_.
As discussed offline: Can we make the comment here a little clearer? It should
state that everything in this block dies when the graph zone is deleted.

https://codereview.chromium.org/662513005/diff/100001/src/compiler/pipeline.c...
src/compiler/pipeline.cc:148: 
Likewise we should add a comment here stating that everything in this block dies
when the instruction zone is deleted.

https://codereview.chromium.org/662513005/diff/100001/src/compiler/pipeline.h
File src/compiler/pipeline.h (right):

https://codereview.chromium.org/662513005/diff/100001/src/compiler/pipeline.h...
src/compiler/pipeline.h:29: class PipelineData;
nit: Please alpha-sort.

[dcarney, 2014-10-27 09:37:04]

Committed patchset #4 (id:140001) manually as 24894 (presubmit successful).