git
/
gcc
mirror of git://gcc.gnu.org/git/gcc.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

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
This commit is contained in:
Jakub Jelinek 2009-09-30 11:57:56 +02:00 committed by Jakub Jelinek
parent 3c8348d376
commit 53a51cef4c
2 changed files with 12 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
gcc/ChangeLog
View File

@ -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> 2009-09-30 Nick Clifton <nickc@redhat.com>
* config.gcc (sh-symbianelf): Replace definition of extra_objs * config.gcc (sh-symbianelf): Replace definition of extra_objs

8
gcc/cfgloopanal.c
View File

@ -175,12 +175,14 @@ num_loop_insns (const struct loop *loop)
for (i = 0; i < loop->num_nodes; i++) for (i = 0; i < loop->num_nodes; i++)
{ {
bb = bbs[i]; bb = bbs[i];
ninsns++;
FOR_BB_INSNS (bb, insn) FOR_BB_INSNS (bb, insn)
if (NONDEBUG_INSN_P (insn)) if (NONDEBUG_INSN_P (insn))
ninsns++; ninsns++;
} }
free(bbs); free (bbs);
if (!ninsns)
ninsns = 1; /* To avoid division by zero. */
return ninsns; return ninsns;
} }
@ -209,7 +211,7 @@ average_num_loop_insns (const struct loop *loop)
: (bb->frequency * BB_FREQ_MAX) / loop->header->frequency; : (bb->frequency * BB_FREQ_MAX) / loop->header->frequency;
ninsns += binsns * ratio; ninsns += binsns * ratio;
} }
free(bbs); free (bbs);
ninsns /= BB_FREQ_MAX; ninsns /= BB_FREQ_MAX;
if (!ninsns) if (!ninsns)