[turbofan] reenable: extend escape analysis to reduce CheckMaps

[turbofan] reenable: extend escape analysis to reduce CheckMaps

R=bmeurer@chromium.org

Review-Url: https://codereview.chromium.org/2803643004
Cr-Commit-Position: refs/heads/master@{#44461}
Committed: https://chromium.googlesource.com/v8/v8/+/25aff3448eb1813702d098361437a7e88256a1c1

[Tobias Tebbi, 2017-04-06 13:45:17]

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

https://codereview.chromium.org/2803643004/diff/1/src/compiler/escape-analysi...
src/compiler/escape-analysis.cc:1532: if (FLAG_turbo_experimental) {
Why doesn't it ignore whitespace? This CL really just deletes this line...

Alternatively, I could also switch the flag, it is only used here.

[Benedikt Meurer, 2017-04-06 14:33:51]

LGTM! Let's see what happens, and act only if we need to act.

[Tobias Tebbi, 2017-04-06 20:16:29]

The CQ bit was checked by tebbi@chromium.org

[commit-bot: I haz the power, 2017-04-06 20:16:48]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-04-06 21:02:20]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1491509789822420, "parent_rev":
"b115095ce113fb62da11d7fabbf220953cbf7746", "commit_rev":
"25aff3448eb1813702d098361437a7e88256a1c1"}

[commit-bot: I haz the power, 2017-04-06 21:02:30]

Description was changed from

==========
[turbofan] reenable: extend escape analysis to reduce CheckMaps

R=bmeurer@chromium.org
==========

to

==========
[turbofan] reenable: extend escape analysis to reduce CheckMaps

R=bmeurer@chromium.org

Review-Url: https://codereview.chromium.org/2803643004
Cr-Commit-Position: refs/heads/master@{#44461}
Committed:
https://chromium.googlesource.com/v8/v8/+/25aff3448eb1813702d098361437a7e8825...
==========

[commit-bot: I haz the power, 2017-04-06 21:02:30]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/v8/v8/+/25aff3448eb1813702d098361437a7e8825...

[Tobias Tebbi, 2017-04-10 11:04:28]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/2811693002/ by tebbi@chromium.org.

The reason for reverting is: canary crashes.