[turbofan] Remove some dead code in move optimizer.

[turbofan] Remove some dead code in move optimizer.

There was no use for the second temp vector, and the one we
are using is scoped to certain methods, and that scope can be
further restricted.

I'm curious if there is really any value in having the temp
vector instead of allocating a function-scoped local. Will
verify that separately.

Committed: https://crrev.com/d33b8f20d5e44f3e6cc7b86c6b02d5f64f20994b
Cr-Commit-Position: refs/heads/master@{#32998}

[Mircea Trofin, 2015-12-22 02:18:23]

Patchset #1 (id:1) has been deleted

[Mircea Trofin, 2015-12-22 02:18:29]

Patchset #1 (id:20001) has been deleted

[Mircea Trofin, 2015-12-22 02:18:35]

Patchset #1 (id:40001) has been deleted

[Mircea Trofin, 2015-12-22 02:21:14]

Description was changed from

==========
gaps can move over any instruction that doesn't feature the operands in the gap.

BUG=
==========

to

==========
There was no use for the second temp vector, and the one we
are using is scoped to certain methods, and that scope can be
further restricted. 

I'm curious if there is really any value in having the temp 
vector instead of allocating a function-scoped local. Will
verify that separately.

BUG=
==========

[Mircea Trofin, 2015-12-22 02:21:15]

mtrofin@chromium.org changed reviewers:
+ bmeurer@chromium.org, jarin@chromium.org

[Benedikt Meurer, 2015-12-22 05:53:54]

LGTM.

[Benedikt Meurer, 2015-12-22 05:54:15]

Description was changed from

==========
There was no use for the second temp vector, and the one we
are using is scoped to certain methods, and that scope can be
further restricted. 

I'm curious if there is really any value in having the temp 
vector instead of allocating a function-scoped local. Will
verify that separately.

BUG=
==========

to

==========
[turbofan] Remove some dead code in move optimizer.

There was no use for the second temp vector, and the one we
are using is scoped to certain methods, and that scope can be
further restricted. 

I'm curious if there is really any value in having the temp 
vector instead of allocating a function-scoped local. Will
verify that separately.
==========

[Mircea Trofin, 2015-12-22 06:36:38]

The CQ bit was checked by mtrofin@chromium.org

[commit-bot: I haz the power, 2015-12-22 06:36:47]

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

[commit-bot: I haz the power, 2015-12-22 06:37:55]

Description was changed from

==========
[turbofan] Remove some dead code in move optimizer.

There was no use for the second temp vector, and the one we
are using is scoped to certain methods, and that scope can be
further restricted. 

I'm curious if there is really any value in having the temp 
vector instead of allocating a function-scoped local. Will
verify that separately.
==========

to

==========
[turbofan] Remove some dead code in move optimizer.

There was no use for the second temp vector, and the one we
are using is scoped to certain methods, and that scope can be
further restricted. 

I'm curious if there is really any value in having the temp 
vector instead of allocating a function-scoped local. Will
verify that separately.
==========

[commit-bot: I haz the power, 2015-12-22 06:37:56]

Committed patchset #1 (id:60001)

[commit-bot: I haz the power, 2015-12-22 06:38:09]

Description was changed from

==========
[turbofan] Remove some dead code in move optimizer.

There was no use for the second temp vector, and the one we
are using is scoped to certain methods, and that scope can be
further restricted. 

I'm curious if there is really any value in having the temp 
vector instead of allocating a function-scoped local. Will
verify that separately.
==========

to

==========
[turbofan] Remove some dead code in move optimizer.

There was no use for the second temp vector, and the one we
are using is scoped to certain methods, and that scope can be
further restricted.

I'm curious if there is really any value in having the temp
vector instead of allocating a function-scoped local. Will
verify that separately.

Committed: https://crrev.com/d33b8f20d5e44f3e6cc7b86c6b02d5f64f20994b
Cr-Commit-Position: refs/heads/master@{#32998}
==========

[commit-bot: I haz the power, 2015-12-22 06:38:10]

Patchset 1 (id:??) landed as
https://crrev.com/d33b8f20d5e44f3e6cc7b86c6b02d5f64f20994b
Cr-Commit-Position: refs/heads/master@{#32998}