public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* cfg fixup bug fix
@ 2011-08-01  7:13 Xinliang David Li
  2011-08-03 17:11 ` Xinliang David Li
  0 siblings, 1 reply; 2+ messages in thread
From: Xinliang David Li @ 2011-08-01  7:13 UTC (permalink / raw)
  To: GCC Patches; +Cc: Jan Hubicka

[-- Attachment #1: Type: text/plain, Size: 247 bytes --]

The attached patch fixed a minor bug in cfg fixup -- the outgoing edge
profile count is not scaled after inlining leading to warnings printed
in IR dump -- 'Invalid sum of ...'.

Bootstrap and tested on x86-64/linux, ok for trunk?

Thanks,

David

[-- Attachment #2: p2 --]
[-- Type: application/octet-stream, Size: 672 bytes --]

2011-07-31  David Li  <davidxl@google.com>

	* tree-optimize.c (execute_fixup_cfg): Fix up entry outgoing
	edge profile counts.

Index: tree-optimize.c
===================================================================
--- tree-optimize.c	(revision 176943)
+++ tree-optimize.c	(working copy)
@@ -256,6 +256,10 @@ execute_fixup_cfg (void)
   EXIT_BLOCK_PTR->count = (EXIT_BLOCK_PTR->count * count_scale
   			   + REG_BR_PROB_BASE / 2) / REG_BR_PROB_BASE;
 
+  FOR_EACH_EDGE (e, ei, ENTRY_BLOCK_PTR->succs)
+    e->count = (e->count * count_scale
+       + REG_BR_PROB_BASE / 2) / REG_BR_PROB_BASE;
+
   FOR_EACH_BB (bb)
     {
       bb->count = (bb->count * count_scale

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: cfg fixup bug fix
  2011-08-01  7:13 cfg fixup bug fix Xinliang David Li
@ 2011-08-03 17:11 ` Xinliang David Li
  0 siblings, 0 replies; 2+ messages in thread
From: Xinliang David Li @ 2011-08-03 17:11 UTC (permalink / raw)
  To: GCC Patches; +Cc: Jan Hubicka

Ping.

David

On Sun, Jul 31, 2011 at 11:58 PM, Xinliang David Li <davidxl@google.com> wrote:
> The attached patch fixed a minor bug in cfg fixup -- the outgoing edge
> profile count is not scaled after inlining leading to warnings printed
> in IR dump -- 'Invalid sum of ...'.
>
> Bootstrap and tested on x86-64/linux, ok for trunk?
>
> Thanks,
>
> David
>

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2011-08-03 17:11 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-08-01  7:13 cfg fixup bug fix Xinliang David Li
2011-08-03 17:11 ` Xinliang David Li

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).