From 53a51cef4c965ba77f4d0ad2f8242b01cda44a3e Mon Sep 17 00:00:00 2001 From: Jakub Jelinek Date: Wed, 30 Sep 2009 11:57:56 +0200 Subject: [PATCH] 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 --- gcc/ChangeLog | 7 +++++++ gcc/cfgloopanal.c | 8 +++++--- 2 files changed, 12 insertions(+), 3 deletions(-) diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 0c6ebdc0f223..3765cba1aed2 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,10 @@ +2009-09-30 Jakub Jelinek + + 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 * config.gcc (sh-symbianelf): Replace definition of extra_objs diff --git a/gcc/cfgloopanal.c b/gcc/cfgloopanal.c index 33aff6dbcebf..129ec25a331b 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)