diff options
| author | Martin Jambor <mjambor@suse.cz> | 2021-09-16 14:04:06 +0200 |
|---|---|---|
| committer | Martin Jambor <mjambor@suse.cz> | 2021-09-16 17:06:47 +0200 |
| commit | 371848a7ed301ee4d6e33842ba41e79412c5d0b0 (patch) | |
| tree | 52f74423660dd67c5d061db50d6db4a304a0fda0 /gcc | |
| parent | ab08859e37ef822c839bc33ec097091f17ebfe76 (diff) | |
| download | gcc-371848a7ed301ee4d6e33842ba41e79412c5d0b0.zip gcc-371848a7ed301ee4d6e33842ba41e79412c5d0b0.tar.gz gcc-371848a7ed301ee4d6e33842ba41e79412c5d0b0.tar.bz2 | |
cgraph: Do not warn about caller count mismatches of removed functions
To verify other changes in the patch series, I have been searching for
"Invalid sum of caller counts" string in symtab dump but found that
there are false warnings about functions which have their body removed
because they are now unreachable. Those are of course invalid and so
this patches avoids checking such cgraph_nodes.
gcc/ChangeLog:
2021-08-20 Martin Jambor <mjambor@suse.cz>
* cgraph.c (cgraph_node::dump): Do not check caller count sums if
the body has been removed. Remove trailing whitespace.
Diffstat (limited to 'gcc')
| -rw-r--r-- | gcc/cgraph.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/gcc/cgraph.c b/gcc/cgraph.c index 8f3af00..de07865 100644 --- a/gcc/cgraph.c +++ b/gcc/cgraph.c @@ -2236,7 +2236,7 @@ cgraph_node::dump (FILE *f) } fprintf (f, "\n"); - if (count.ipa ().initialized_p ()) + if (!body_removed && count.ipa ().initialized_p ()) { bool ok = true; bool min = false; @@ -2245,7 +2245,7 @@ cgraph_node::dump (FILE *f) FOR_EACH_ALIAS (this, ref) if (dyn_cast <cgraph_node *> (ref->referring)->count.initialized_p ()) sum += dyn_cast <cgraph_node *> (ref->referring)->count.ipa (); - + if (inlined_to || (symtab->state < EXPANSION && ultimate_alias_target () == this && only_called_directly_p ())) |