[turbofan] Solidify OSR deconstruction code a bit.

[turbofan] Solidify OSR deconstruction code a bit.

It is invalid for OSR deconstruction to leave a graph with a node
representing the OSR normal entry (and no OSR loop entry). Subsequent
lowering phases will not handle {OsrNormalEntry} operators and hence
will lead to serious clogging further down the pipeline.

R=bmeurer@chromium.org
BUG=chromium:641893
Committed: https://crrev.com/d2189c63628e55c7eee2e58b9529b7f156b89acd
Cr-Commit-Position: refs/heads/master@{#39340}

[Michael Starzinger, 2016-09-12 09:59:31]

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

[commit-bot: I haz the power, 2016-09-12 09:59:38]

Dry run: 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-12 10:22:01]

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

[commit-bot: I haz the power, 2016-09-12 10:22:02]

Dry run: This issue passed the CQ dry run.

[Michael Starzinger, 2016-09-12 10:41:38]

Description was changed from

==========
[turbofan] Solidify OSR deconstruction code a bit.

It is invalid for OSR deconstruction to leave a graph with a node
representing the OSR normal entry (and no OSR loop entry). Subsequent
lowering phases will not handle {OsrNormalEntry} operators and hence
will lead to serious clogging further down the pipeline.

R=
BUG=chromium:641893
==========

to

==========
[turbofan] Solidify OSR deconstruction code a bit.

It is invalid for OSR deconstruction to leave a graph with a node
representing the OSR normal entry (and no OSR loop entry). Subsequent
lowering phases will not handle {OsrNormalEntry} operators and hence
will lead to serious clogging further down the pipeline.

R=bmeurer@chromium.org
BUG=chromium:641893
==========

[Michael Starzinger, 2016-09-12 10:41:38]

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

[Michael Starzinger, 2016-09-12 10:42:00]

Benedikt: PTAL.
Ben: FYI.

[Benedikt Meurer, 2016-09-12 10:42:45]

Awesome, thanks. That should hopefully detect the bug early. LGTM.

[Michael Starzinger, 2016-09-12 10:44:08]

The CQ bit was checked by mstarzinger@chromium.org

[commit-bot: I haz the power, 2016-09-12 10:44:11]

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-12 10:46:11]

Description was changed from

==========
[turbofan] Solidify OSR deconstruction code a bit.

It is invalid for OSR deconstruction to leave a graph with a node
representing the OSR normal entry (and no OSR loop entry). Subsequent
lowering phases will not handle {OsrNormalEntry} operators and hence
will lead to serious clogging further down the pipeline.

R=bmeurer@chromium.org
BUG=chromium:641893
==========

to

==========
[turbofan] Solidify OSR deconstruction code a bit.

It is invalid for OSR deconstruction to leave a graph with a node
representing the OSR normal entry (and no OSR loop entry). Subsequent
lowering phases will not handle {OsrNormalEntry} operators and hence
will lead to serious clogging further down the pipeline.

R=bmeurer@chromium.org
BUG=chromium:641893
==========

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

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-09-12 10:46:31]

Description was changed from

==========
[turbofan] Solidify OSR deconstruction code a bit.

It is invalid for OSR deconstruction to leave a graph with a node
representing the OSR normal entry (and no OSR loop entry). Subsequent
lowering phases will not handle {OsrNormalEntry} operators and hence
will lead to serious clogging further down the pipeline.

R=bmeurer@chromium.org
BUG=chromium:641893
==========

to

==========
[turbofan] Solidify OSR deconstruction code a bit.

It is invalid for OSR deconstruction to leave a graph with a node
representing the OSR normal entry (and no OSR loop entry). Subsequent
lowering phases will not handle {OsrNormalEntry} operators and hence
will lead to serious clogging further down the pipeline.

R=bmeurer@chromium.org
BUG=chromium:641893

Committed: https://crrev.com/d2189c63628e55c7eee2e58b9529b7f156b89acd
Cr-Commit-Position: refs/heads/master@{#39340}
==========

[commit-bot: I haz the power, 2016-09-12 10:46:32]

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