[crankshaft] Get rid of the StoreFrameContext instruction.

[crankshaft] Get rid of the StoreFrameContext instruction.

Context is always available through deopt data, so there should be
no need to store the context back to the frame every time. (Turbofan
already does not store back to the frame.)

Committed: https://crrev.com/6f75a216f1b709c2fb5fbced5e9deaa61ba3affe
Cr-Commit-Position: refs/heads/master@{#35125}

[Jarin, 2016-03-30 07:06:01]

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

[Jarin, 2016-03-30 07:06:02]

Red only CL; could you take a look, please?

[Benedikt Meurer, 2016-03-30 07:07:56]

https://codereview.chromium.org/1845553002/diff/1/src/crankshaft/hydrogen.cc
File src/crankshaft/hydrogen.cc (left):

https://codereview.chromium.org/1845553002/diff/1/src/crankshaft/hydrogen.cc#...
src/crankshaft/hydrogen.cc:4706: AddSimulate(stmt->EntryId(),
REMOVABLE_SIMULATE);
Please also remove this BailoutId from the Statement if it's never used.

https://codereview.chromium.org/1845553002/diff/1/src/crankshaft/hydrogen.cc#...
src/crankshaft/hydrogen.cc:4725: AddSimulate(stmt->ExitId(),
REMOVABLE_SIMULATE);
Please also remove this BailoutId from the Statement if it's never used.

[Benedikt Meurer, 2016-03-30 07:19:00]

As discussed offline, let's leave it this way for now, and clean it up once we
kill FCG.

[Benedikt Meurer, 2016-03-30 07:19:07]

LGTM

[Jarin, 2016-03-30 07:19:32]

The CQ bit was checked by jarin@chromium.org

[commit-bot: I haz the power, 2016-03-30 07:19:34]

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

[commit-bot: I haz the power, 2016-03-30 07:21:23]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-03-30 07:23:15]

Description was changed from

==========
[crankshaft] Get rid of the StoreFrameContext instruction.

Context is always available through deopt data, so there should be
no need to store the context back to the frame every time. (Turbofan
already does not store back to the frame.)
==========

to

==========
[crankshaft] Get rid of the StoreFrameContext instruction.

Context is always available through deopt data, so there should be
no need to store the context back to the frame every time. (Turbofan
already does not store back to the frame.)

Committed: https://crrev.com/6f75a216f1b709c2fb5fbced5e9deaa61ba3affe
Cr-Commit-Position: refs/heads/master@{#35125}
==========

[commit-bot: I haz the power, 2016-03-30 07:23:16]

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