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
> course everything after it, including the combiner.
>
> Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk?
Seems obvious to me? Okay for trunk as far as combine is concerned :-)
Segher
> 2019-01-10 Jakub Jelinek <ja...@redhat.com>
>
> PR bootstrap/88714
> * passes.c (finish_optimization_passes): Call print_combine_total_stats
> inside of pass_combine_1 dump rather than pass_profile_1.
>
> --- gcc/passes.c.jj 2019-01-01 12:37:15.494002253 +0100
> +++ gcc/passes.c 2019-01-10 16:30:43.295424173 +0100
> @@ -361,9 +361,9 @@ finish_optimization_passes (void)
>
> if (optimize > 0)
> {
> - dumps->dump_start (pass_profile_1->static_pass_number, NULL);
> + dumps->dump_start (pass_combine_1->static_pass_number, NULL);
> print_combine_total_stats ();
> - dumps->dump_finish (pass_profile_1->static_pass_number);
> + dumps->dump_finish (pass_combine_1->static_pass_number);
> }
>
> /* Do whatever is necessary to finish printing the graphs. */
>
