[turbofan] fixed escape analysis bug: missing copy of virtual state

[turbofan] fixed escape analysis bug: missing copy of virtual state

R=jarin@chromium.org

BUG=chromium:673243
Review-Url: https://codereview.chromium.org/2578133002
Cr-Commit-Position: refs/heads/master@{#41848}
Committed: https://chromium.googlesource.com/v8/v8/+/abd63018d7db329b3e702c01799d5136d600bcfa

[Tobias Tebbi, 2016-12-15 14:15:11]

Patchset #1 (id:1) has been deleted

[Tobias Tebbi, 2016-12-15 14:16:17]

Patchset #1 (id:20001) has been deleted

[Jarin, 2016-12-15 14:30:10]

Great catch!

Please add the test.

Also use "BUG=chromium:nnnnnn" rather than "BUG=nnnnnn" so that it automatically
updates the bug.

[Tobias Tebbi, 2016-12-15 14:52:29]

Description was changed from

==========
[turbofan] fixed escape analysis bug: missing copy of virtual state

R=jarin@chromium.org

BUG=673243
==========

to

==========
[turbofan] fixed escape analysis bug: missing copy of virtual state

R=jarin@chromium.org

BUG=chromium:673243
==========

[Jarin, 2016-12-19 07:28:44]

lgtm

https://codereview.chromium.org/2578133002/diff/60001/test/mjsunit/compiler/e...
File test/mjsunit/compiler/escape-analysis-framestate-use-at-branchpoint.js
(right):

https://codereview.chromium.org/2578133002/diff/60001/test/mjsunit/compiler/e...
test/mjsunit/compiler/escape-analysis-framestate-use-at-branchpoint.js:1: //
Copyright 2013 the V8 project authors. All rights reserved.
Nit: wrong year, unnecessarily long header.

https://codereview.chromium.org/2578133002/diff/60001/test/mjsunit/compiler/e...
test/mjsunit/compiler/escape-analysis-framestate-use-at-branchpoint.js:39:
foo();
nit: Funny indentation.

[Tobias Tebbi, 2016-12-20 10:02:48]

The CQ bit was checked by tebbi@chromium.org

[Tobias Tebbi, 2016-12-20 10:02:49]

The patchset sent to the CQ was uploaded after l-g-t-m from jarin@chromium.org
Link to the patchset: https://codereview.chromium.org/2578133002/#ps80001
(title: "addressed comments")

[commit-bot: I haz the power, 2016-12-20 10:02:58]

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, 2016-12-20 10:30:53]

CQ is committing da patch.

Bot data: {"patchset_id": 80001, "attempt_start_ts": 1482228168806190,
"parent_rev": "3bede2f79d325dadfd4fabe950feaf4122ede339", "commit_rev":
"abd63018d7db329b3e702c01799d5136d600bcfa"}

[commit-bot: I haz the power, 2016-12-20 10:30:56]

Description was changed from

==========
[turbofan] fixed escape analysis bug: missing copy of virtual state

R=jarin@chromium.org

BUG=chromium:673243
==========

to

==========
[turbofan] fixed escape analysis bug: missing copy of virtual state

R=jarin@chromium.org

BUG=chromium:673243

Review-Url: https://codereview.chromium.org/2578133002
Cr-Commit-Position: refs/heads/master@{#41848}
Committed:
https://chromium.googlesource.com/v8/v8/+/abd63018d7db329b3e702c01799d5136d60...
==========

[commit-bot: I haz the power, 2016-12-20 10:30:57]

Committed patchset #3 (id:80001) as
https://chromium.googlesource.com/v8/v8/+/abd63018d7db329b3e702c01799d5136d60...