diff options
author | Jakub Jelinek <jakub@redhat.com> | 2009-09-30 11:57:56 +0200 |
---|---|---|
committer | Jakub Jelinek <jakub@gcc.gnu.org> | 2009-09-30 11:57:56 +0200 |
commit | 53a51cef4c965ba77f4d0ad2f8242b01cda44a3e (patch) | |
tree | 4e22c3671349439e180deee055e799d2e1c28b03 /gcc/cfgloopanal.c | |
parent | 3c8348d376afad91fb9f89f14960af14e8cd5e77 (diff) | |
download | gcc-53a51cef4c965ba77f4d0ad2f8242b01cda44a3e.tar.gz |
re PR target/41279 (252.eon performance regression)
PR target/41279
* cfgloopanal.c (num_loop_insns): Don't increment ninsns for each bb
before insn counting loop now that BB_END (bb) is counted. Ensure
the return value isn't zero.
From-SVN: r152324
Diffstat (limited to 'gcc/cfgloopanal.c')
-rw-r--r-- | gcc/cfgloopanal.c | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/gcc/cfgloopanal.c b/gcc/cfgloopanal.c index 33aff6dbceb..129ec25a331 100644 --- a/gcc/cfgloopanal.c +++ b/gcc/cfgloopanal.c @@ -175,12 +175,14 @@ num_loop_insns (const struct loop *loop) for (i = 0; i < loop->num_nodes; i++) { bb = bbs[i]; - ninsns++; FOR_BB_INSNS (bb, insn) if (NONDEBUG_INSN_P (insn)) ninsns++; } - free(bbs); + free (bbs); + + if (!ninsns) + ninsns = 1; /* To avoid division by zero. */ return ninsns; } @@ -209,7 +211,7 @@ average_num_loop_insns (const struct loop *loop) : (bb->frequency * BB_FREQ_MAX) / loop->header->frequency; ninsns += binsns * ratio; } - free(bbs); + free (bbs); ninsns /= BB_FREQ_MAX; if (!ninsns) |