> You are right, we need discriminator for non-CALL stmts too. Patch updated:
OK for google branches. Thanks!
-cary
You are right, we need discriminator for non-CALL stmts too. Patch updated:
Index: gcc/tree-cfg.c
===
--- gcc/tree-cfg.c (revision 201858)
+++ gcc/tree-cfg.c (working copy)
@@ -781,9 +781,37 @@ assign_discriminators (void)
{
> This patch assigns discriminators for different callsites within the
> same BB. This is needed for accurate profile attribution in AutoFDO.
>
> Testing on going.
>
> OK for google branches if test pass?
> + for (gsi = gsi_start_bb (bb); !gsi_end_p (gsi); gsi_next (&gsi))
> + {
> + gimple s
All tests pass.
Dehao
On Mon, Aug 19, 2013 at 3:02 PM, Dehao Chen wrote:
> This patch assigns discriminators for different callsites within the
> same BB. This is needed for accurate profile attribution in AutoFDO.
>
> Testing on going.
>
> OK for google branches if test pass?
>
> Thanks,
> Deha
This patch assigns discriminators for different callsites within the
same BB. This is needed for accurate profile attribution in AutoFDO.
Testing on going.
OK for google branches if test pass?
Thanks,
Dehao
Index: gcc/tree-cfg.c
==
