From: Jason Merrill Date: Thu, 15 Oct 2020 16:54:16 +0000 (-0400) Subject: c++: Fix [[deprecated]] and implicit operator==. [PR97358] X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=f3ee94724686b82556c07b4d33821ae973eb9aba;p=gcc.git c++: Fix [[deprecated]] and implicit operator==. [PR97358] Trying to diagnose the problem with an implicit copy function breaks if the function isn't actually a copy function. gcc/cp/ChangeLog: PR c++/95844 * decl.c (copy_fn_p): Return false for a function that is neither a constructor nor an assignment operator. (move_signature_fn_p): Likewise. gcc/testsuite/ChangeLog: PR c++/95844 * g++.dg/cpp2a/spaceship-eq10.C: New test. --- diff --git a/gcc/cp/decl.c b/gcc/cp/decl.c index 2f1a2f0c554..5f370e60b4e 100644 --- a/gcc/cp/decl.c +++ b/gcc/cp/decl.c @@ -14169,6 +14169,10 @@ copy_fn_p (const_tree d) accept those as copy functions. */ return 0; + if (!DECL_CONSTRUCTOR_P (d) + && DECL_NAME (d) != assign_op_identifier) + return 0; + args = FUNCTION_FIRST_USER_PARMTYPE (d); if (!args) return 0; @@ -14242,6 +14246,10 @@ move_signature_fn_p (const_tree d) tree arg_type; bool result = false; + if (!DECL_CONSTRUCTOR_P (d) + && DECL_NAME (d) != assign_op_identifier) + return 0; + args = FUNCTION_FIRST_USER_PARMTYPE (d); if (!args) return 0; diff --git a/gcc/testsuite/g++.dg/cpp2a/spaceship-eq10.C b/gcc/testsuite/g++.dg/cpp2a/spaceship-eq10.C new file mode 100644 index 00000000000..93f5e253af4 --- /dev/null +++ b/gcc/testsuite/g++.dg/cpp2a/spaceship-eq10.C @@ -0,0 +1,11 @@ +// PR c++/95844 +// { dg-do compile { target c++20 } } + +#include + +struct F { + [[deprecated("oh no")]] std::strong_ordering operator<=>(const F&) const = default; // { dg-message "" } +}; +void use_f(F f) { + void(f == f); // { dg-warning "deprecated" } +}