[turbofan] Do not use the state value cache when building the tree

[turbofan] Do not use the state value cache when building the tree

Since the liveness analysis's non-live value clearing rebuilds the state
value trees, we don't need to be smart when creating state values in the
initial graph building. This simplifies both the building and the
iteration over the state values by the liveness analyzer.

Committed: https://crrev.com/4a660091ee284630c2d315f8a0e736675bec8855
Cr-Commit-Position: refs/heads/master@{#40996}

[Leszek Swirski, 2016-11-15 13:14:56]

The CQ bit was checked by leszeks@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-11-15 13:15:09]

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

[Leszek Swirski, 2016-11-15 13:15:38]

leszeks@chromium.org changed reviewers:
+ jarin@chromium.org

[Leszek Swirski, 2016-11-15 13:15:39]

As discussed f2f, I see ~2% improvements on Box2D and Typescript with this.

[Jarin, 2016-11-15 13:25:06]

On 2016/11/15 13:15:39, Leszek Swirski wrote:
> As discussed f2f, I see ~2% improvements on Box2D and Typescript with this.

Could you upload again? This patch is somehow corrupted.

[commit-bot: I haz the power, 2016-11-15 13:42:02]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-11-15 13:42:03]

Dry run: This issue passed the CQ dry run.

[Leszek Swirski, 2016-11-15 13:55:58]

On 2016/11/15 13:25:06, Jarin wrote:
> On 2016/11/15 13:15:39, Leszek Swirski wrote:
> > As discussed f2f, I see ~2% improvements on Box2D and Typescript with this.
> 
> Could you upload again? This patch is somehow corrupted.

Weird. Done.

[Leszek Swirski, 2016-11-15 13:58:09]

The CQ bit was checked by leszeks@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-11-15 13:58:12]

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

[Jarin, 2016-11-15 14:06:08]

lgtm. Thanks!

[Leszek Swirski, 2016-11-15 14:08:28]

The CQ bit was unchecked by leszeks@chromium.org

[Leszek Swirski, 2016-11-15 14:08:28]

The CQ bit was checked by leszeks@chromium.org

[commit-bot: I haz the power, 2016-11-15 14:08:36]

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-11-15 14:22:55]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-11-17 22:34:15]

Description was changed from

==========
[turbofan] Do not use the state value cache when building the tree

Since the liveness analysis's non-live value clearing rebuilds the state
value trees, we don't need to be smart when creating state values in the
initial graph building. This simplifies both the building and the
iteration over the state values by the liveness analyzer.
==========

to

==========
[turbofan] Do not use the state value cache when building the tree

Since the liveness analysis's non-live value clearing rebuilds the state
value trees, we don't need to be smart when creating state values in the
initial graph building. This simplifies both the building and the
iteration over the state values by the liveness analyzer.

Committed: https://crrev.com/4a660091ee284630c2d315f8a0e736675bec8855
Cr-Commit-Position: refs/heads/master@{#40996}
==========

[commit-bot: I haz the power, 2016-11-17 22:34:16]

Patchset 2 (id:??) landed as
https://crrev.com/4a660091ee284630c2d315f8a0e736675bec8855
Cr-Commit-Position: refs/heads/master@{#40996}