[turbofan] Fix corner case in loop analysis.

[turbofan] Fix corner case in loop analysis.

R=mstarzinger@chromium.org
BUG=

Committed: https://crrev.com/3244a66543ef2ab20eb41c1adce1d9d0a5744929
Cr-Commit-Position: refs/heads/master@{#26063}

[Michael Starzinger, 2015-01-14 16:28:32]

LGTM with one nit.

https://codereview.chromium.org/852783002/diff/1/src/compiler/loop-analysis.cc
File src/compiler/loop-analysis.cc (right):

https://codereview.chromium.org/852783002/diff/1/src/compiler/loop-analysis.c...
src/compiler/loop-analysis.cc:191: NodeInfo& CreateLoopInfo(Node* node,
NodeInfo& ni) {
nit: Instead of returning the NodeInfo here, we could just keep this "void" and
store the NodeInfo in a local variable at the call-site in line 164. I think
that would be easier to read.

[titzer, 2015-01-14 17:06:01]

On 2015/01/14 16:28:32, Michael Starzinger wrote:
> LGTM with one nit.
> 
> https://codereview.chromium.org/852783002/diff/1/src/compiler/loop-analysis.cc
> File src/compiler/loop-analysis.cc (right):
> 
>
https://codereview.chromium.org/852783002/diff/1/src/compiler/loop-analysis.c...
> src/compiler/loop-analysis.cc:191: NodeInfo& CreateLoopInfo(Node* node,
> NodeInfo& ni) {
> nit: Instead of returning the NodeInfo here, we could just keep this "void"
and
> store the NodeInfo in a local variable at the call-site in line 164. I think
> that would be easier to read.

Done, but it took two lines! :(

[titzer, 2015-01-14 17:06:44]

Committed patchset #2 (id:20001) manually as
3244a66543ef2ab20eb41c1adce1d9d0a5744929 (presubmit successful).

[commit-bot: I haz the power, 2015-01-14 17:06:45]

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