[turbofan] Remove unsafe JSToBoolean lowering.

[turbofan] Remove unsafe JSToBoolean lowering.

The lowering of {JSToBoolean} operators in {JSTypedLowering} inserts
loads that are not part of the effect chain. This does not play well
with effect-sensitive data flow analysis (e.g. escape analysis). This
removes the lowering in question, we can implement it using a dedicated
simplified operator eventually if needed.

R=bmeurer@chromium.org
TEST=mjsunit/wasm/embenchen/lua_binarytrees

Committed: https://crrev.com/66e73b3a1b898787d3a3f79f4135e0047f296d2a
Cr-Commit-Position: refs/heads/master@{#39773}

[Michael Starzinger, 2016-09-27 12:05:53]

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

[commit-bot: I haz the power, 2016-09-27 12:05:55]

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

[Benedikt Meurer, 2016-09-27 12:17:28]

LGTM

[commit-bot: I haz the power, 2016-09-27 12:21:25]

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

[commit-bot: I haz the power, 2016-09-27 12:21:26]

Dry run: Try jobs failed on following builders:
  v8_linux64_gyp_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_gyp_rel_ng/build...)
  v8_linux64_gyp_rel_ng_triggered on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_gyp_rel_ng_trigg...)

[Michael Starzinger, 2016-09-27 13:00:28]

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

[commit-bot: I haz the power, 2016-09-27 13:00:32]

Dry run: 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-09-27 13:49:11]

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

[commit-bot: I haz the power, 2016-09-27 13:49:12]

Dry run: This issue passed the CQ dry run.

[Michael Starzinger, 2016-09-27 13:52:24]

The CQ bit was checked by mstarzinger@chromium.org

[Michael Starzinger, 2016-09-27 13:52:24]

The patchset sent to the CQ was uploaded after l-g-t-m from bmeurer@chromium.org
Link to the patchset: https://codereview.chromium.org/2366363003/#ps20001
(title: "Remove unit test.")

[commit-bot: I haz the power, 2016-09-27 13:52:30]

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-09-27 13:55:09]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-09-27 13:55:33]

Description was changed from

==========
[turbofan] Remove unsafe JSToBoolean lowering.

The lowering of {JSToBoolean} operators in {JSTypedLowering} inserts
loads that are not part of the effect chain. This does not play well
with effect-sensitive data flow analysis (e.g. escape analysis). This
removes the lowering in question, we can implement it using a dedicated
simplified operator eventually if needed.

R=bmeurer@chromium.org
TEST=mjsunit/wasm/embenchen/lua_binarytrees
==========

to

==========
[turbofan] Remove unsafe JSToBoolean lowering.

The lowering of {JSToBoolean} operators in {JSTypedLowering} inserts
loads that are not part of the effect chain. This does not play well
with effect-sensitive data flow analysis (e.g. escape analysis). This
removes the lowering in question, we can implement it using a dedicated
simplified operator eventually if needed.

R=bmeurer@chromium.org
TEST=mjsunit/wasm/embenchen/lua_binarytrees

Committed: https://crrev.com/66e73b3a1b898787d3a3f79f4135e0047f296d2a
Cr-Commit-Position: refs/heads/master@{#39773}
==========

[commit-bot: I haz the power, 2016-09-27 13:55:34]

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