procyberian/gcc
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

tree-ssa-loop-niter.c (estimate_numbers_of_iterations_loop): Avoid use of profile unless profile status is PROFILE_READ.

Browse source 
* tree-ssa-loop-niter.c (estimate_numbers_of_iterations_loop): Avoid
	use of profile unless profile status is PROFILE_READ.
	* profile.c (compute_branch_probabilities): Set profile status
	only after reporting predictor hitrates.

From-SVN: r237076
This commit is contained in:
Jan Hubicka 2016-06-03 19:00:19 +02:00 • committed by Jan Hubicka
parent 0d2f700f7b
commit 641762ae26
3 changed files with 19 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
gcc/ChangeLog
View file

@ -1,3 +1,10 @@
2016-06-03 Jan Hubicka <hubicka@ucw.cz>
* tree-ssa-loop-niter.c (estimate_numbers_of_iterations_loop): Avoid
use of profile unless profile status is PROFILE_READ.
* profile.c (compute_branch_probabilities): Set profile status
only after reporting predictor hitrates.
2016-06-03 Joseph Myers <joseph@codesourcery.com>
PR target/71276

11
gcc/profile.c
View file

@ -826,8 +826,6 @@ compute_branch_probabilities (unsigned cfg_checksum, unsigned lineno_checksum)
}
}
counts_to_freqs ();
profile_status_for_fn (cfun) = PROFILE_READ;
compute_function_frequency ();
if (dump_file)
{
@ -1329,8 +1327,13 @@ branch_prob (void)
values.release ();
free_edge_list (el);
coverage_end_function (lineno_checksum, cfg_checksum);
if (dump_file && (dump_flags & TDF_DETAILS))
report_predictor_hitrates ();
if (flag_branch_probabilities && profile_info)
{
if (dump_file && (dump_flags & TDF_DETAILS))
report_predictor_hitrates ();
profile_status_for_fn (cfun) = PROFILE_READ;
compute_function_frequency ();
}
}
/* Union find algorithm implementation for the basic blocks using

8
gcc/tree-ssa-loop-niter.c
View file

@ -3757,10 +3757,12 @@ estimate_numbers_of_iterations_loop (struct loop *loop)
maybe_lower_iteration_bound (loop);
/* If we have a measured profile, use it to estimate the number of
iterations. */
if (loop->header->count != 0)
iterations. Explicitly check for profile status so we do not report
wrong prediction hitrates for guessed loop iterations heuristics. */
if (loop->header->count != 0
&& profile_status_for_fn (cfun) >= PROFILE_READ)
{
gcov_type nit = expected_loop_iterations_unbounded (loop) + 1;
gcov_type nit = expected_loop_iterations_unbounded (loop);
bound = gcov_type_to_wide_int (nit);
record_niter_bound (loop, bound, true, false);
}