[turbofan] Always defer replacement in simplified lowering.

[turbofan] Always defer replacement in simplified lowering.

Since we do negative type checks, the current shortcut might still affect
the lowering choices, possibly leading to inconsistent results.

This CL gets rid of the shortcut.

Committed: https://crrev.com/10714b633cbba472b1e68e6063a6d89ee903f5e6
Cr-Commit-Position: refs/heads/master@{#37412}

[Jarin, 2016-06-29 12:04:28]

Description was changed from

==========
[turbofan] Always defer replacement in deferred lowering.

Since we do negative type checks, the current shortcut might still affect
the lowering choices, possibly leading to inconsistent results.
==========

to

==========
[turbofan] Always defer replacement in simplified lowering.

Since we do negative type checks, the current shortcut might still affect
the lowering choices, possibly leading to inconsistent results.

This CL gets rid of the shortcut.
==========

[Jarin, 2016-06-29 12:04:29]

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

[Jarin, 2016-06-29 12:06:34]

Could you take a look, please?

As far as I can see, this fixes the Richards flaky crashes - the unpatched
version had 2 crashes per 1000 runs, the patched version did not crash.

[Benedikt Meurer, 2016-06-29 12:37:53]

LGTM, thanks.

[Jarin, 2016-06-30 05:36:15]

The CQ bit was checked by jarin@chromium.org

[commit-bot: I haz the power, 2016-06-30 05:36:26]

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-06-30 05:38:29]

Description was changed from

==========
[turbofan] Always defer replacement in simplified lowering.

Since we do negative type checks, the current shortcut might still affect
the lowering choices, possibly leading to inconsistent results.

This CL gets rid of the shortcut.
==========

to

==========
[turbofan] Always defer replacement in simplified lowering.

Since we do negative type checks, the current shortcut might still affect
the lowering choices, possibly leading to inconsistent results.

This CL gets rid of the shortcut.
==========

[commit-bot: I haz the power, 2016-06-30 05:38:30]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-06-30 05:39:23]

Description was changed from

==========
[turbofan] Always defer replacement in simplified lowering.

Since we do negative type checks, the current shortcut might still affect
the lowering choices, possibly leading to inconsistent results.

This CL gets rid of the shortcut.
==========

to

==========
[turbofan] Always defer replacement in simplified lowering.

Since we do negative type checks, the current shortcut might still affect
the lowering choices, possibly leading to inconsistent results.

This CL gets rid of the shortcut.

Committed: https://crrev.com/10714b633cbba472b1e68e6063a6d89ee903f5e6
Cr-Commit-Position: refs/heads/master@{#37412}
==========

[commit-bot: I haz the power, 2016-06-30 05:39:24]

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