typeck.c (cp_build_function_call_vec): When mark_used fails unconditionally return error_mark_node.

/cp
2019-05-11  Paolo Carlini  <paolo.carlini@oracle.com>

	* typeck.c (cp_build_function_call_vec): When mark_used fails
	unconditionally return error_mark_node.

/testsuite
2019-05-11  Paolo Carlini  <paolo.carlini@oracle.com>

	* g++.dg/cpp2a/multiple-deleted-destroying-delete-error-1.C: New.
	* g++.dg/cpp2a/multiple-deleted-destroying-delete-error-2.C: Likewise.

From-SVN: r271091

gcc/cp/ChangeLog

+2019-05-11  Paolo Carlini  <paolo.carlini@oracle.com>
+
+	* typeck.c (cp_build_function_call_vec): When mark_used fails
+	unconditionally return error_mark_node.
+
 2019-05-10  Paolo Carlini  <paolo.carlini@oracle.com>
 
 	* decl.c (grokvardecl): Use an accurate location in error message

gcc/cp/typeck.c

@@ -3836,7 +3836,7 @@ cp_build_function_call_vec (tree function, vec<tree, va_gc> **params,
           return error_mark_node;
         }
 
-      if (!mark_used (function, complain) && !(complain & tf_error))
+      if (!mark_used (function, complain))
 	return error_mark_node;
       fndecl = function;
 

gcc/testsuite/ChangeLog

+2019-05-11  Paolo Carlini  <paolo.carlini@oracle.com>
+
+	* g++.dg/cpp2a/multiple-deleted-destroying-delete-error-1.C: New.
+	* g++.dg/cpp2a/multiple-deleted-destroying-delete-error-2.C: Likewise.
+
 2019-05-10  Thomas Koenig  <tkoenig@gcc.gnu.org>
 
 	PR fortran/61968

gcc/testsuite/g++.dg/cpp2a/multiple-deleted-destroying-delete-error-1.C

+// { dg-do compile { target c++2a } }
+
+#include <new>
+
+namespace delete_selection_d {
+  struct B {
+    void operator delete(void*) = delete;
+    void operator delete(B *, std::destroying_delete_t) = delete;
+  };
+  void delete_B(B *b) { delete b; }  // { dg-bogus "deleted .* deleted" }
+  // { dg-error "deleted" "" { target c++2a } .-1 }
+}

gcc/testsuite/g++.dg/cpp2a/multiple-deleted-destroying-delete-error-2.C

+// { dg-do compile { target c++2a } }
+
+#include <new>
+
+namespace delete_selection_r {
+  struct B {
+    void operator delete(B *, std::destroying_delete_t) = delete;
+    void operator delete(void*) = delete;
+  };
+  void delete_B(B *b) { delete b; }  // { dg-bogus "deleted .* deleted" }
+  // { dg-error "deleted" "" { target c++2a } .-1 }
+}