[turbofan] refactor pipeline to use hydrogen like Run calls

[turbofan] refactor pipeline to use hydrogen like Run calls

BUG=

Committed: https://chromium.googlesource.com/v8/v8/+/2738466f3b60009d835129090434e9b66211acaa

[titzer, 2014-11-14 14:03:39]

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

[titzer, 2014-11-14 14:03:40]

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

https://codereview.chromium.org/727733002/diff/60001/src/compiler/pipeline.cc...
src/compiler/pipeline.cc:258: void Run(PipelineData* data) {
I like this a lot better!

Can we pull the phase_scope out to the Run() that calls this and also the
zone_scope, and then pass in the zone_scope?

[dcarney, 2014-11-14 15:48:46]

more, ptal

[titzer, 2014-11-14 16:07:47]

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

https://codereview.chromium.org/727733002/diff/100001/src/compiler/pipeline.c...
src/compiler/pipeline.cc:330: static const char* phase_name() { return "graph
builder"; }
You can just make these static fields.

[titzer, 2014-11-14 16:13:55]

lgtm

[dcarney, 2014-11-14 16:16:28]

The CQ bit was checked by dcarney@chromium.org

[commit-bot: I haz the power, 2014-11-14 16:17:15]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/727733002/140001

[commit-bot: I haz the power, 2014-11-14 16:44:49]

Committed patchset #8 (id:140001)