[gcc(refs/users/guojiufu/heads/guojiufu-branch)] ipa: Various diagnostic fixes [PR93797]
Jiu Fu Guo
guojiufu@gcc.gnu.org
Wed Mar 11 02:04:03 GMT 2020
https://gcc.gnu.org/g:da67227bfc407dd1412b3396ed962f30084f9401
commit da67227bfc407dd1412b3396ed962f30084f9401
Author: Jakub Jelinek <jakub@redhat.com>
Date: Tue Feb 18 09:54:17 2020 +0100
ipa: Various diagnostic fixes [PR93797]
As the patch shows, various messages didn't match the field names they are
talking about.
2020-02-18 Jakub Jelinek <jakub@redhat.com>
PR ipa/93797
* cgraph.c (verify_speculative_call): Use speculative_id instead of
speculative_uid in messages. Remove trailing whitespace from error
message. Use num_speculative_call_targets instead of
num_speculative_targets in a message.
(cgraph_node::verify_node): Use call_stmt instead of cal_stmt in
edge messages and stmt instead of cal_stmt in reference message.
Diff:
---
gcc/ChangeLog | 8 ++++++++
gcc/cgraph.c | 18 +++++++++---------
2 files changed, 17 insertions(+), 9 deletions(-)
diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index 99d32c04bfb..efc21166d6a 100644
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,5 +1,13 @@
2020-02-18 Jakub Jelinek <jakub@redhat.com>
+ PR ipa/93797
+ * cgraph.c (verify_speculative_call): Use speculative_id instead of
+ speculative_uid in messages. Remove trailing whitespace from error
+ message. Use num_speculative_call_targets instead of
+ num_speculative_targets in a message.
+ (cgraph_node::verify_node): Use call_stmt instead of cal_stmt in
+ edge messages and stmt instead of cal_stmt in reference message.
+
PR tree-optimization/93780
* tree-ssa.c (non_rewritable_lvalue_p): Check valid_vector_subparts_p
before calling build_vector_type.
diff --git a/gcc/cgraph.c b/gcc/cgraph.c
index 294b2d392fd..c420863a0fd 100644
--- a/gcc/cgraph.c
+++ b/gcc/cgraph.c
@@ -3230,14 +3230,14 @@ verify_speculative_call (struct cgraph_node *node, gimple *stmt,
if (direct->speculative_id >= num)
{
error ("direct call to %s in speculative call sequence has "
- "speculative_uid %i out of range",
+ "speculative_id %i out of range",
direct->callee->dump_name (), direct->speculative_id);
return true;
}
if (direct_calls[direct->speculative_id])
{
error ("duplicate direct call to %s in speculative call sequence "
- "with speculative_uid %i",
+ "with speculative_id %i",
direct->callee->dump_name (), direct->speculative_id);
return true;
}
@@ -3248,7 +3248,7 @@ verify_speculative_call (struct cgraph_node *node, gimple *stmt,
&& first_call != node->get_edge (first_call->call_stmt))
{
error ("call stmt hash does not point to first direct edge of "
- "speculative call sequence ");
+ "speculative call sequence");
return true;
}
@@ -3260,14 +3260,14 @@ verify_speculative_call (struct cgraph_node *node, gimple *stmt,
if (ref->speculative_id >= num)
{
error ("direct call to %s in speculative call sequence has "
- "speculative_uid %i out of range",
+ "speculative_id %i out of range",
ref->referred->dump_name (), ref->speculative_id);
return true;
}
if (refs[ref->speculative_id])
{
error ("duplicate reference %s in speculative call sequence "
- "with speculative_uid %i",
+ "with speculative_id %i",
ref->referred->dump_name (), ref->speculative_id);
return true;
}
@@ -3294,7 +3294,7 @@ verify_speculative_call (struct cgraph_node *node, gimple *stmt,
if (num_targets != indirect->num_speculative_call_targets_p ())
{
error ("number of speculative targets %i mismatched with "
- "num_speculative_targets %i",
+ "num_speculative_call_targets %i",
num_targets,
indirect->num_speculative_call_targets_p ());
return true;
@@ -3400,7 +3400,7 @@ cgraph_node::verify_node (void)
}
if (e->call_stmt && e->lto_stmt_uid)
{
- error ("edge has both cal_stmt and lto_stmt_uid set");
+ error ("edge has both call_stmt and lto_stmt_uid set");
error_found = true;
}
}
@@ -3471,7 +3471,7 @@ cgraph_node::verify_node (void)
}
if (e->call_stmt && e->lto_stmt_uid)
{
- error ("edge has both cal_stmt and lto_stmt_uid set");
+ error ("edge has both call_stmt and lto_stmt_uid set");
error_found = true;
}
if (e->speculative
@@ -3509,7 +3509,7 @@ cgraph_node::verify_node (void)
{
if (ref->stmt && ref->lto_stmt_uid)
{
- error ("reference has both cal_stmt and lto_stmt_uid set");
+ error ("reference has both stmt and lto_stmt_uid set");
error_found = true;
}
if (ref->speculative
More information about the Gcc-cvs
mailing list