[turbofan] Address the useless overflow bit materialization.

[turbofan] Address the useless overflow bit materialization.

Add control dependencies to Projection and Int32Add/SubWithOverflow
operators, to prevent the scheduler from moving the Projection nodes
into the wrong place. This way the instruction selection can combine
the Int32Add/SubWithOverflow operations with the DeoptimizeIf and/or
DeoptimizeUnless nodes. This needs new operators CheckedInt32Add and
CheckedInt32Sub so that we can delay the actual lowering until the
effect/control linearizer.

This also makes CheckIf operator obsolete, so we can drop it.

R=jarin@chromium.org

Committed: https://crrev.com/00889cc29c1489a8d00b4bfbef47b48efc45af5d
Cr-Commit-Position: refs/heads/master@{#37148}

[Jarin, 2016-06-21 12:32:42]

lgtm

[Benedikt Meurer, 2016-06-21 12:42:12]

The CQ bit was checked by bmeurer@chromium.org

[commit-bot: I haz the power, 2016-06-21 12:42:20]

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

[commit-bot: I haz the power, 2016-06-21 14:26:33]

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

[commit-bot: I haz the power, 2016-06-21 14:26:35]

Try jobs failed on following builders:
  v8_linux64_asan_rel_ng on tryserver.v8 (JOB_TIMED_OUT, no build URL)

[Benedikt Meurer (Google), 2016-06-21 15:30:27]

The CQ bit was checked by bmeurer@google.com

[commit-bot: I haz the power, 2016-06-21 15:30:34]

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

[commit-bot: I haz the power, 2016-06-21 15:46:21]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-06-21 15:46:48]

Description was changed from

==========
[turbofan] Address the useless overflow bit materialization.

Add control dependencies to Projection and Int32Add/SubWithOverflow
operators, to prevent the scheduler from moving the Projection nodes
into the wrong place. This way the instruction selection can combine
the Int32Add/SubWithOverflow operations with the DeoptimizeIf and/or
DeoptimizeUnless nodes. This needs new operators CheckedInt32Add and
CheckedInt32Sub so that we can delay the actual lowering until the
effect/control linearizer.

This also makes CheckIf operator obsolete, so we can drop it.

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

to

==========
[turbofan] Address the useless overflow bit materialization.

Add control dependencies to Projection and Int32Add/SubWithOverflow
operators, to prevent the scheduler from moving the Projection nodes
into the wrong place. This way the instruction selection can combine
the Int32Add/SubWithOverflow operations with the DeoptimizeIf and/or
DeoptimizeUnless nodes. This needs new operators CheckedInt32Add and
CheckedInt32Sub so that we can delay the actual lowering until the
effect/control linearizer.

This also makes CheckIf operator obsolete, so we can drop it.

R=jarin@chromium.org

Committed: https://crrev.com/00889cc29c1489a8d00b4bfbef47b48efc45af5d
Cr-Commit-Position: refs/heads/master@{#37148}
==========

[commit-bot: I haz the power, 2016-06-21 15:46:49]

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