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 | |
| parent | 3c8348d376afad91fb9f89f14960af14e8cd5e77 (diff) | |
| download | gcc-53a51cef4c965ba77f4d0ad2f8242b01cda44a3e.zip gcc-53a51cef4c965ba77f4d0ad2f8242b01cda44a3e.tar.gz gcc-53a51cef4c965ba77f4d0ad2f8242b01cda44a3e.tar.bz2 | |
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
| -rw-r--r-- | gcc/ChangeLog | 7 | ||||
| -rw-r--r-- | gcc/cfgloopanal.c | 8 |
2 files changed, 12 insertions, 3 deletions
diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 0c6ebdc..3765cba 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,10 @@ +2009-09-30 Jakub Jelinek <jakub@redhat.com> + + 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. + 2009-09-30 Nick Clifton <nickc@redhat.com> * config.gcc (sh-symbianelf): Replace definition of extra_objs diff --git a/gcc/cfgloopanal.c b/gcc/cfgloopanal.c index 33aff6db..129ec25 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) |