public inbox for gcc-cvs@sourceware.org
help / color / mirror / Atom feed
* [gcc/devel/c++-coroutines] ipa: Various diagnostic fixes [PR93797]
@ 2020-02-18 20:44 Iain D Sandoe
0 siblings, 0 replies; only message in thread
From: Iain D Sandoe @ 2020-02-18 20:44 UTC (permalink / raw)
To: gcc-cvs
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 99d32c0..efc2116 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 294b2d3..c420863 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
^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2020-02-18 20:44 UTC | newest]
Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-02-18 20:44 [gcc/devel/c++-coroutines] ipa: Various diagnostic fixes [PR93797] Iain D Sandoe
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).