[turbofan] Fix loop assignment analysis on ForInStatements.

[turbofan] Fix loop assignment analysis on ForInStatements.

The implicit assignment to the induction variable in a ForInStatement
has been ignored by the AST loop assignment analysis. This was hidden
for cases where the parser introduced a ".for" temporary, but triggers
when the variable is declared outside the loop.

R=bmeurer@chromium.org
TEST=mjsunit/regress/regress-crbug-647887
BUG=chromium:647887
Committed: https://crrev.com/4dab7b5a1d6722002d47d0be2481cb65602a2451
Cr-Commit-Position: refs/heads/master@{#39551}

[Michael Starzinger, 2016-09-20 11:42:57]

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

[commit-bot: I haz the power, 2016-09-20 11:43:06]

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

[Michael Starzinger, 2016-09-20 11:43:34]

mstarzinger@chromium.org changed reviewers:
+ titzer@chromium.org

[Michael Starzinger, 2016-09-20 11:43:34]

Benedikt: PTAL.
Ben: FYI.

[titzer, 2016-09-20 11:52:23]

lgtm

[commit-bot: I haz the power, 2016-09-20 12:18:45]

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

[commit-bot: I haz the power, 2016-09-20 12:18:46]

Dry run: This issue passed the CQ dry run.

[Benedikt Meurer, 2016-09-20 12:28:40]

lgtm

[Michael Starzinger, 2016-09-20 12:34:29]

The CQ bit was checked by mstarzinger@chromium.org

[commit-bot: I haz the power, 2016-09-20 12:34:37]

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-20 12:36:58]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-09-20 12:37:43]

Description was changed from

==========
[turbofan] Fix loop assignment analysis on ForInStatements.

The implicit assignment to the induction variable in a ForInStatement
has been ignored by the AST loop assignment analysis. This was hidden
for cases where the parser introduced a ".for" temporary, but triggers
when the variable is declared outside the loop.

R=bmeurer@chromium.org
TEST=mjsunit/regress/regress-crbug-647887
BUG=chromium:647887
==========

to

==========
[turbofan] Fix loop assignment analysis on ForInStatements.

The implicit assignment to the induction variable in a ForInStatement
has been ignored by the AST loop assignment analysis. This was hidden
for cases where the parser introduced a ".for" temporary, but triggers
when the variable is declared outside the loop.

R=bmeurer@chromium.org
TEST=mjsunit/regress/regress-crbug-647887
BUG=chromium:647887

Committed: https://crrev.com/4dab7b5a1d6722002d47d0be2481cb65602a2451
Cr-Commit-Position: refs/heads/master@{#39551}
==========

[commit-bot: I haz the power, 2016-09-20 12:37:44]

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