[turbofan] Improve loop analysis to handle more than 32 loops.

[turbofan] Improve loop analysis to handle more than 32 loops.

R=mstarzinger@chromium.org
BUG=

Committed: https://crrev.com/36003d07eb650d8c2dddae95ad136aac17c83c0c
Cr-Commit-Position: refs/heads/master@{#26140}

[Michael Starzinger, 2015-01-19 12:51:21]

LGTM. Stared long and hard at the matrix index computations, couldn't find any
bugs, but I am sure they are hiding in there.

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

https://codereview.chromium.org/855653002/diff/1/src/compiler/loop-analysis.c...
src/compiler/loop-analysis.cc:17: #define OFFSET(x) ((x)&0x1f)
nit: Whitespace before and after operator for readability.

https://codereview.chromium.org/855653002/diff/1/test/cctest/compiler/test-lo...
File test/cctest/compiler/test-loop-analysis.cc (right):

https://codereview.chromium.org/855653002/diff/1/test/cctest/compiler/test-lo...
test/cctest/compiler/test-loop-analysis.cc:962: TEST(LaManyChained_31) {
RunManyChainedLoops_i(30); }
nit: s/30/31/ here.

https://codereview.chromium.org/855653002/diff/1/test/cctest/compiler/test-lo...
test/cctest/compiler/test-loop-analysis.cc:971: TEST(LaManyNested_31) {
RunManyNestedLoops_i(30); }
nit: s/30/31/ here.

[titzer, 2015-01-19 12:54:49]

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

https://codereview.chromium.org/855653002/diff/1/src/compiler/loop-analysis.c...
src/compiler/loop-analysis.cc:17: #define OFFSET(x) ((x)&0x1f)
On 2015/01/19 12:51:20, Michael Starzinger wrote:
> nit: Whitespace before and after operator for readability.

Done.

https://codereview.chromium.org/855653002/diff/1/test/cctest/compiler/test-lo...
File test/cctest/compiler/test-loop-analysis.cc (right):

https://codereview.chromium.org/855653002/diff/1/test/cctest/compiler/test-lo...
test/cctest/compiler/test-loop-analysis.cc:962: TEST(LaManyChained_31) {
RunManyChainedLoops_i(30); }
On 2015/01/19 12:51:20, Michael Starzinger wrote:
> nit: s/30/31/ here.

Done.

https://codereview.chromium.org/855653002/diff/1/test/cctest/compiler/test-lo...
test/cctest/compiler/test-loop-analysis.cc:971: TEST(LaManyNested_31) {
RunManyNestedLoops_i(30); }
On 2015/01/19 12:51:20, Michael Starzinger wrote:
> nit: s/30/31/ here.

Done.

[titzer, 2015-01-19 12:54:59]

The CQ bit was checked by titzer@chromium.org

[titzer, 2015-01-19 12:55:42]

The CQ bit was unchecked by titzer@chromium.org

[titzer, 2015-01-19 12:55:58]

The CQ bit was checked by titzer@chromium.org

[commit-bot: I haz the power, 2015-01-19 12:56:56]

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

[commit-bot: I haz the power, 2015-01-19 13:54:37]

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

[commit-bot: I haz the power, 2015-01-19 13:54:38]

Try jobs failed on following builders:
  v8_win64_rel on tryserver.v8
(http://build.chromium.org/p/tryserver.v8/builders/v8_win64_rel/builds/1253)

[titzer, 2015-01-19 14:05:41]

The CQ bit was checked by titzer@chromium.org

[commit-bot: I haz the power, 2015-01-19 14:06:36]

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

[commit-bot: I haz the power, 2015-01-19 14:25:18]

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

[commit-bot: I haz the power, 2015-01-19 14:25:18]

Try jobs failed on following builders:
  v8_win64_rel on tryserver.v8
(http://build.chromium.org/p/tryserver.v8/builders/v8_win64_rel/builds/1256)

[titzer, 2015-01-19 14:51:47]

The CQ bit was checked by titzer@chromium.org

[commit-bot: I haz the power, 2015-01-19 14:52:16]

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

[commit-bot: I haz the power, 2015-01-19 15:19:51]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2015-01-19 15:20:07]

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