[turbofan] Remove the EmptyFrameState caching on JSGraph.

[turbofan] Remove the EmptyFrameState caching on JSGraph.

Caching nodes with mutable inputs is a bad idea and already blew up
twice now, so in order to avoid further breakage, let's kill the
EmptyFrameState caching on JSGraph completely and only cache the empty
state values there.

We can remove the hacking from JSTypedLowering completely once we have
the PlainPrimitiveToNumber in action.

R=jarin@chromium.org

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

[Benedikt Meurer, 2016-05-25 10:43:16]

Hey Jaro,

The caching is potentially always broken as long as you have mutable inputs. So
use the caching only for nodes with no inputs.
Please take a look.

Thanks,
Benedikt

[Jarin, 2016-05-25 10:50:12]

lgtm

[Benedikt Meurer, 2016-05-25 10:50:33]

The CQ bit was checked by bmeurer@chromium.org

[commit-bot: I haz the power, 2016-05-25 10:50:45]

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

[commit-bot: I haz the power, 2016-05-25 11:04:49]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-05-25 11:05:19]

Description was changed from

==========
[turbofan] Remove the EmptyFrameState caching on JSGraph.

Caching nodes with mutable inputs is a bad idea and already blew up
twice now, so in order to avoid further breakage, let's kill the
EmptyFrameState caching on JSGraph completely and only cache the empty
state values there.

We can remove the hacking from JSTypedLowering completely once we have
the PlainPrimitiveToNumber in action.

R=jarin@chromium.org
==========

to

==========
[turbofan] Remove the EmptyFrameState caching on JSGraph.

Caching nodes with mutable inputs is a bad idea and already blew up
twice now, so in order to avoid further breakage, let's kill the
EmptyFrameState caching on JSGraph completely and only cache the empty
state values there.

We can remove the hacking from JSTypedLowering completely once we have
the PlainPrimitiveToNumber in action.

R=jarin@chromium.org

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

[commit-bot: I haz the power, 2016-05-25 11:05:20]

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