[gcc/devel/c++-coroutines] ipa: Various diagnostic fixes [PR93797]

[gcc/devel/c++-coroutines] ipa: Various diagnostic fixes [PR93797]

[Iain D Sandoe]

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