Refactor Scheduler to simplify construction of CFG from sea of nodes. Use PreEdge/Post as part of t…

Refactor Scheduler to simplify construction of CFG from sea of nodes. Use PreEdge/Post as part of the graph tarversal, and make finding of successor projections more robust.

R=jarin@chromium.org
BUG=

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

[titzer, 2014-08-19 14:07:34]

PTAL

[Jarin, 2014-08-19 14:37:28]

https://codereview.chromium.org/490483002/diff/1/src/compiler/scheduler.cc
File src/compiler/scheduler.cc (right):

https://codereview.chromium.org/490483002/diff/1/src/compiler/scheduler.cc#ne...
src/compiler/scheduler.cc:20: if (FLAG_trace_turbo_scheduler) PrintF x
Please, do not use this macro. It could lead to very hard-to-find bugs.

If your really want shorter tracing invocations, do the same thing as
RegisterAllocator::TraceAlloc.

[titzer, 2014-08-19 15:17:23]

On 2014/08/19 14:37:28, jarin wrote:
> https://codereview.chromium.org/490483002/diff/1/src/compiler/scheduler.cc
> File src/compiler/scheduler.cc (right):
> 
>
https://codereview.chromium.org/490483002/diff/1/src/compiler/scheduler.cc#ne...
> src/compiler/scheduler.cc:20: if (FLAG_trace_turbo_scheduler) PrintF x
> Please, do not use this macro. It could lead to very hard-to-find bugs.
> 
> If your really want shorter tracing invocations, do the same thing as
> RegisterAllocator::TraceAlloc.

Done.

[Jarin, 2014-08-19 19:01:55]

lgtm. Thanks!

[titzer, 2014-08-20 08:57:04]

Committed patchset #2 manually as 23217 (presubmit successful).