Re: [PATCH] Fix misplaced combine totals dumping (PR bootstrap/88714)

2019-01-11 Thread Richard Biener
On January 10, 2019 11:25:59 PM GMT+01:00, Jakub Jelinek wrote: >Hi! > >r191883 seems to have introduced a pasto: >--- trunk/gcc/passes.c 2012/10/01 00:17:52 191882 >+++ trunk/gcc/passes.c 2012/10/01 05:43:06 191883 >@@ -231,27 +231,23 @@ > timevar_push (TV_DUMP); >if (profile_arc_flag

Re: [PATCH] Fix misplaced combine totals dumping (PR bootstrap/88714)

2019-01-11 Thread Segher Boessenkool
On Thu, Jan 10, 2019 at 11:25:59PM +0100, Jakub Jelinek wrote: > So, in the end, the combiner statistics was emitted in profile_estimate dump > and on the PR88714 issue suggested there is a difference already in the > profile_estimate dump, when actually the IL changed only during pre and of > cour

[PATCH] Fix misplaced combine totals dumping (PR bootstrap/88714)

2019-01-10 Thread Jakub Jelinek
Hi! r191883 seems to have introduced a pasto: --- trunk/gcc/passes.c 2012/10/01 00:17:52 191882 +++ trunk/gcc/passes.c 2012/10/01 05:43:06 191883 @@ -231,27 +231,23 @@ timevar_push (TV_DUMP); if (profile_arc_flag || flag_test_coverage || flag_branch_probabilities) { - dum