[turbofan] The PlainPrimitiveToNumber operator cannot have side effects.

[turbofan] The PlainPrimitiveToNumber operator cannot have side effects.

An invocation of the ToNumber builtin resulting from lowering the
PlainPrimitiveToNumber cannot have any observable side effects and
thus the builtin call itself is eliminatable.

R=yangguo@chromium.org

Committed: https://crrev.com/63dd16849f5cf2b7c9c21dc073d6d470273b016c
Cr-Commit-Position: refs/heads/master@{#38593}

[Benedikt Meurer, 2016-08-12 04:34:19]

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

[commit-bot: I haz the power, 2016-08-12 04:34:28]

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

[Yang, 2016-08-12 04:52:13]

On 2016/08/12 04:34:18, Benedikt Meurer wrote:

lgtm.

[Benedikt Meurer, 2016-08-12 04:52:42]

The CQ bit was unchecked by bmeurer@chromium.org

[Benedikt Meurer, 2016-08-12 04:52:49]

The CQ bit was checked by bmeurer@chromium.org

[commit-bot: I haz the power, 2016-08-12 04:52:54]

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-08-12 05:08:27]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-12 05:08:53]

Description was changed from

==========
[turbofan] The PlainPrimitiveToNumber operator cannot have side effects.

An invocation of the ToNumber builtin resulting from lowering the
PlainPrimitiveToNumber cannot have any observable side effects and
thus the builtin call itself is eliminatable.

R=yangguo@chromium.org
==========

to

==========
[turbofan] The PlainPrimitiveToNumber operator cannot have side effects.

An invocation of the ToNumber builtin resulting from lowering the
PlainPrimitiveToNumber cannot have any observable side effects and
thus the builtin call itself is eliminatable.

R=yangguo@chromium.org

Committed: https://crrev.com/63dd16849f5cf2b7c9c21dc073d6d470273b016c
Cr-Commit-Position: refs/heads/master@{#38593}
==========

[commit-bot: I haz the power, 2016-08-12 05:08:54]

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