[turbofan] Reduce duplication between ControlReducer::ReduceIf(True,False).

[turbofan] Reduce duplication between ControlReducer::ReduceIf(True,False).

R=svenpanne@chromium.org
BUG=

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

[Sven Panne, 2015-04-02 09:34:05]

LGTM with a tiny nit.

https://codereview.chromium.org/1053583005/diff/1/src/compiler/control-reduce...
File src/compiler/control-reducer.cc (right):

https://codereview.chromium.org/1053583005/diff/1/src/compiler/control-reduce...
src/compiler/control-reducer.cc:585: // fold a true branch by replacing
IfTrue/IfFalse with the branch control.
Nit: The words "true branch" are perhaps not perfect anymore, but I fail to see
a truly better formulation. Perhaps "branch which is always taken"? Don't
know... Feel free to improve or leave it as it is, it's blindingly obvious
either way. :-D

[titzer, 2015-04-02 09:37:52]

On 2015/04/02 09:34:05, Sven Panne wrote:
> LGTM with a tiny nit.
> 
>
https://codereview.chromium.org/1053583005/diff/1/src/compiler/control-reduce...
> File src/compiler/control-reducer.cc (right):
> 
>
https://codereview.chromium.org/1053583005/diff/1/src/compiler/control-reduce...
> src/compiler/control-reducer.cc:585: // fold a true branch by replacing
> IfTrue/IfFalse with the branch control.
> Nit: The words "true branch" are perhaps not perfect anymore, but I fail to
see
> a truly better formulation. Perhaps "branch which is always taken"? Don't
> know... Feel free to improve or leave it as it is, it's blindingly obvious
> either way. :-D

I reworded the sentence slightly.

[titzer, 2015-04-02 09:38:25]

The CQ bit was checked by titzer@chromium.org

[titzer, 2015-04-02 09:38:26]

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

[commit-bot: I haz the power, 2015-04-02 09:38:35]

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

[commit-bot: I haz the power, 2015-04-02 10:03:19]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-04-02 10:03:37]

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