* typeck.c (check_return_expr): Avoid redundant error.

From-SVN: r277798

gcc/cp/ChangeLog

+2019-11-04  Jason Merrill  <jason@redhat.com>
+
+	* typeck.c (check_return_expr): Avoid redundant error.
+
 2019-11-02  Paolo Carlini  <paolo.carlini@oracle.com>
 
 	* typeck.c (composite_pointer_type): Add a const op_location_t&

gcc/cp/typeck.c

@@ -9729,7 +9729,7 @@ check_return_expr (tree retval, bool *no_warning)
 	   type.  In that case, we have to evaluate the expression for
 	   its side-effects.  */
 	finish_expr_stmt (retval);
-      else
+      else if (retval != error_mark_node)
 	permerror (input_location,
 		   "return-statement with a value, in function "
 		   "returning %qT", valtype);

gcc/testsuite/g++.dg/other/return2.C

+void f(long);
+void f(char);
+
+void g()
+{
+  return f(42);			// { dg-error "ambiguous" }
+}				// { dg-bogus "void" "" { target *-*-* } .-1 }