[wasm] Break a cycle with a loop node in the int64-lowering.

[wasm] Break a cycle with a loop node in the int64-lowering.

The control edges in a TurboFan graph can form a cycle. To break this cycle in the int64-lowering we add special handling for loop nodes. Similar handling already exists for phi nodes and effectphi nodes, which breaks cycles formed by value edges and effect edges, respectively.

Committed: https://crrev.com/020e2c8d267217a789b80fabf460ebaf37ca964d
Cr-Commit-Position: refs/heads/master@{#41071}

[ahaas, 2016-11-16 19:38:42]

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

[commit-bot: I haz the power, 2016-11-16 19:38:49]

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-11-16 20:08:02]

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

[commit-bot: I haz the power, 2016-11-16 20:08:03]

Dry run: This issue passed the CQ dry run.

[ahaas, 2016-11-17 10:37:09]

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

[commit-bot: I haz the power, 2016-11-17 10:37:14]

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

[ahaas, 2016-11-17 10:41:42]

Description was changed from

==========
[wasm] Break a cycle with a loop node in the int64-lowering.
==========

to

==========
[wasm] Break a cycle with a loop node in the int64-lowering.

The control edges in a TurboFan graph can form a cycle. To break this cycle in
the int64-lowering we add special handling for loop nodes. Similar handling
already exists for phi nodes and effectphi nodes, which breaks cycles formed by
value edges and effect edges, respectively.
==========

[ahaas, 2016-11-17 10:41:42]

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

[commit-bot: I haz the power, 2016-11-17 10:53:05]

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

[commit-bot: I haz the power, 2016-11-17 10:53:06]

Dry run: Try jobs failed on following builders:
  v8_linux_dbg_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_dbg_ng/builds/16413)
  v8_linux_dbg_ng_triggered on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_dbg_ng_triggered/b...)

[titzer, 2016-11-17 10:55:44]

On 2016/11/17 10:54:46, ahaas wrote:

lgtm

[ahaas, 2016-11-17 11:27:12]

The CQ bit was checked by ahaas@chromium.org

[commit-bot: I haz the power, 2016-11-17 11:27:16]

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-11-17 11:50:42]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-11-17 22:38:03]

Description was changed from

==========
[wasm] Break a cycle with a loop node in the int64-lowering.

The control edges in a TurboFan graph can form a cycle. To break this cycle in
the int64-lowering we add special handling for loop nodes. Similar handling
already exists for phi nodes and effectphi nodes, which breaks cycles formed by
value edges and effect edges, respectively.
==========

to

==========
[wasm] Break a cycle with a loop node in the int64-lowering.

The control edges in a TurboFan graph can form a cycle. To break this cycle in
the int64-lowering we add special handling for loop nodes. Similar handling
already exists for phi nodes and effectphi nodes, which breaks cycles formed by
value edges and effect edges, respectively.

Committed: https://crrev.com/020e2c8d267217a789b80fabf460ebaf37ca964d
Cr-Commit-Position: refs/heads/master@{#41071}
==========

[commit-bot: I haz the power, 2016-11-17 22:38:04]

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