OpenE2K/gcc
2
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Fix comments typo

Browse Source 
gcc/ChangeLog:

	2019-11-15  Luo Xiong Hu  <luoxhu@linux.ibm.com>

	* ipa-comdats.c: Fix comments typo.
	* ipa-profile.c: Fix comments typo.
	* tree-profile.c (gimple_gen_ic_profiler): Use the new variable
	__gcov_indirect_call.counters and __gcov_indirect_call.callee.
	(gimple_gen_ic_func_profiler): Likewise.
	(pass_ipa_tree_profile::gate): Fix comments typo.

From-SVN: r278278
This commit is contained in:
Luo Xiong Hu 2019-11-15 08:17:31 +00:00 committed by Xiong Hu Luo
parent 2aae99f7a4
commit 53dd3bccac
4 changed files with 17 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
gcc/ChangeLog
View File

@ -1,3 +1,12 @@
2019-11-15 Luo Xiong Hu <luoxhu@linux.ibm.com>
* ipa-comdats.c: Fix comments typo.
* ipa-profile.c: Fix comments typo.
* tree-profile.c (gimple_gen_ic_profiler): Use the new variable
__gcov_indirect_call.counters and __gcov_indirect_call.callee.
(gimple_gen_ic_func_profiler): Likewise.
(pass_ipa_tree_profile::gate): Fix comments typo.
2019-11-15 Xiong Hu Luo <luoxhu@linux.ibm.com> 2019-11-15 Xiong Hu Luo <luoxhu@linux.ibm.com>
* ipa-inline.c (inline_small_functions): Update iterator of next. * ipa-inline.c (inline_small_functions): Update iterator of next.

2
gcc/ipa-comdats.c
View File

@ -18,7 +18,7 @@ along with GCC; see the file COPYING3. If not see
<http://www.gnu.org/licenses/>. */ <http://www.gnu.org/licenses/>. */
/* This is very simple pass that looks for static symbols that are used /* This is very simple pass that looks for static symbols that are used
exlusively by symbol within one comdat group. In this case it makes exclusively by symbol within one comdat group. In this case it makes
sense to bring the symbol itself into the group to avoid dead code sense to bring the symbol itself into the group to avoid dead code
that would arrise when the comdat group from current unit is replaced that would arrise when the comdat group from current unit is replaced
by a different copy. Consider for example: by a different copy. Consider for example:

2
gcc/ipa-profile.c
View File

@ -563,7 +563,7 @@ ipa_profile (void)
histogram.release (); histogram.release ();
histogram_pool.release (); histogram_pool.release ();
/* Produce speculative calls: we saved common traget from porfiling into /* Produce speculative calls: we saved common target from porfiling into
e->common_target_id. Now, at link time, we can look up corresponding e->common_target_id. Now, at link time, we can look up corresponding
function node and produce speculative call. */ function node and produce speculative call. */

12
gcc/tree-profile.c
View File

@ -73,8 +73,8 @@ static GTY(()) tree ic_tuple_callee_field;
/* Do initialization work for the edge profiler. */ /* Do initialization work for the edge profiler. */
/* Add code: /* Add code:
__thread gcov* __gcov_indirect_call_counters; // pointer to actual counter __thread gcov* __gcov_indirect_call.counters; // pointer to actual counter
__thread void* __gcov_indirect_call_callee; // actual callee address __thread void* __gcov_indirect_call.callee; // actual callee address
__thread int __gcov_function_counter; // time profiler function counter __thread int __gcov_function_counter; // time profiler function counter
*/ */
static void static void
@ -381,7 +381,7 @@ gimple_gen_ic_profiler (histogram_value value, unsigned tag)
f_1 = foo; f_1 = foo;
__gcov_indirect_call.counters = &__gcov4.main[0]; __gcov_indirect_call.counters = &__gcov4.main[0];
PROF_9 = f_1; PROF_9 = f_1;
__gcov_indirect_call_callee = PROF_9; __gcov_indirect_call.callee = PROF_9;
_4 = f_1 (); _4 = f_1 ();
*/ */
@ -444,11 +444,11 @@ gimple_gen_ic_func_profiler (void)
/* Insert code: /* Insert code:
if (__gcov_indirect_call_callee != NULL) if (__gcov_indirect_call.callee != NULL)
__gcov_indirect_call_profiler_v3 (profile_id, &current_function_decl); __gcov_indirect_call_profiler_v3 (profile_id, &current_function_decl);
The function __gcov_indirect_call_profiler_v3 is responsible for The function __gcov_indirect_call_profiler_v3 is responsible for
resetting __gcov_indirect_call_callee to NULL. */ resetting __gcov_indirect_call.callee to NULL. */
gimple_stmt_iterator gsi = gsi_start_bb (cond_bb); gimple_stmt_iterator gsi = gsi_start_bb (cond_bb);
void0 = build_int_cst (ptr_type_node, 0); void0 = build_int_cst (ptr_type_node, 0);
@ -890,7 +890,7 @@ pass_ipa_tree_profile::gate (function *)
{ {
/* When profile instrumentation, use or test coverage shall be performed. /* When profile instrumentation, use or test coverage shall be performed.
But for AutoFDO, this there is no instrumentation, thus this pass is But for AutoFDO, this there is no instrumentation, thus this pass is
diabled. */ disabled. */
return (!in_lto_p && !flag_auto_profile return (!in_lto_p && !flag_auto_profile
&& (flag_branch_probabilities || flag_test_coverage && (flag_branch_probabilities || flag_test_coverage
|| profile_arc_flag)); || profile_arc_flag));