[turbofan] Delay initialization in escape analysis

[turbofan] Delay initialization in escape analysis

This is also a refactoring of the merge function in
escape analysis.

BUG=v8:4586
LOG=n

Committed: https://crrev.com/5aa72daa88dda209e69bbff83f6d8c9a585996fa
Cr-Commit-Position: refs/heads/master@{#33728}

[sigurds, 2016-02-02 11:37:04]

Description was changed from

==========
[turbofan] Delay initialization in escape analysis

BUG=v8:4586
LOG=n
==========

to

==========
[turbofan] Delay initialization in escape analysis

BUG=v8:4586
LOG=n
==========

[sigurds, 2016-02-02 11:37:05]

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

[sigurds, 2016-02-02 11:37:14]

PTAL

https://codereview.chromium.org/1654163003/diff/1/src/compiler/escape-analysi...
File src/compiler/escape-analysis.cc (left):

https://codereview.chromium.org/1654163003/diff/1/src/compiler/escape-analysi...
src/compiler/escape-analysis.cc:474: status_(graph->NodeCount(), kUnknown,
zone),
This initialization happens in EscapeAnalysis::Run() anyway. By removing it
here, it does not happen if the graph has no kAllocate nodes (this saves
time+space).

[Michael Starzinger, 2016-02-04 10:10:18]

LGTM.

https://codereview.chromium.org/1654163003/diff/80001/src/compiler/escape-ana...
File src/compiler/escape-analysis.cc (right):

https://codereview.chromium.org/1654163003/diff/80001/src/compiler/escape-ana...
src/compiler/escape-analysis.cc:772: if (FLAG_trace_turbo_escape) {
nit: This can be achieve more easily by moving the respective flag into the
"debug only" section of the flag-definitions.h file.

[sigurds, 2016-02-04 10:35:29]

Description was changed from

==========
[turbofan] Delay initialization in escape analysis

BUG=v8:4586
LOG=n
==========

to

==========
[turbofan] Delay initialization in escape analysis

This is also a refactoring of the merge function in
escape analysis.

BUG=v8:4586
LOG=n
==========

[sigurds, 2016-02-04 11:22:36]

The CQ bit was checked by sigurds@chromium.org

[sigurds, 2016-02-04 11:22:36]

The patchset sent to the CQ was uploaded after l-g-t-m from
mstarzinger@chromium.org
Link to the patchset: https://codereview.chromium.org/1654163003/#ps100001
(title: "Make --trace-turbo-escape a proper debug flag")

[commit-bot: I haz the power, 2016-02-04 11:22:41]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1654163003/100001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1654163003/100001

[commit-bot: I haz the power, 2016-02-04 11:24:12]

Description was changed from

==========
[turbofan] Delay initialization in escape analysis

This is also a refactoring of the merge function in
escape analysis.

BUG=v8:4586
LOG=n
==========

to

==========
[turbofan] Delay initialization in escape analysis

This is also a refactoring of the merge function in
escape analysis.

BUG=v8:4586
LOG=n
==========

[commit-bot: I haz the power, 2016-02-04 11:24:13]

Committed patchset #6 (id:100001)

[commit-bot: I haz the power, 2016-02-04 11:24:42]

Description was changed from

==========
[turbofan] Delay initialization in escape analysis

This is also a refactoring of the merge function in
escape analysis.

BUG=v8:4586
LOG=n
==========

to

==========
[turbofan] Delay initialization in escape analysis

This is also a refactoring of the merge function in
escape analysis.

BUG=v8:4586
LOG=n

Committed: https://crrev.com/5aa72daa88dda209e69bbff83f6d8c9a585996fa
Cr-Commit-Position: refs/heads/master@{#33728}
==========

[commit-bot: I haz the power, 2016-02-04 11:24:43]

Patchset 6 (id:??) landed as
https://crrev.com/5aa72daa88dda209e69bbff83f6d8c9a585996fa
Cr-Commit-Position: refs/heads/master@{#33728}