]> git.ipfire.org Git - thirdparty/gcc.git/commitdiff
c++: P0848R3 and member function templates [PR95181]
authorPatrick Palka <ppalka@redhat.com>
Fri, 22 May 2020 14:28:19 +0000 (10:28 -0400)
committerJason Merrill <jason@redhat.com>
Sat, 30 May 2020 02:11:10 +0000 (22:11 -0400)
When comparing two special member function templates to see if one hides
the other (as per P0848R3), we need to check satisfaction which we can't
do on templates.  So this patch makes add_method skip the eligibility
test on member function templates and just lets them coexist.

gcc/cp/ChangeLog:

PR c++/95181
* class.c (add_method): Let special member function templates
coexist if they are not equivalently constrained, or in a class
template.

gcc/testsuite/ChangeLog:

PR c++/95181
* g++.dg/concepts/pr95181.C: New test.
* g++.dg/concepts/pr95181-2.C: New test.

Co-authored-by: Jason Merrill <jason@redhat.com>
(cherry picked from commit 6b449b74c590f5a6f66c73aed894e5b5b36aa59d)

gcc/cp/class.c
gcc/testsuite/g++.dg/concepts/pr95181-2.C [new file with mode: 0644]
gcc/testsuite/g++.dg/concepts/pr95181.C [new file with mode: 0644]

index 1f524a319174f77e02a271a1eecec215787cc188..5b52a2a1f0a6c1ac05ca293e0ad02c5de0bf5bc7 100644 (file)
@@ -1077,12 +1077,19 @@ add_method (tree type, tree method, bool via_using)
        {
           if (!equivalently_constrained (fn, method))
            {
+             if (processing_template_decl)
+               /* We can't check satisfaction in dependent context, wait until
+                  the class is instantiated.  */
+               continue;
+
              special_function_kind sfk = special_memfn_p (method);
 
-             if (sfk == sfk_none || DECL_INHERITED_CTOR (fn))
-               /* Non-special member functions coexist if they are not
-                  equivalently constrained.  A member function is not hidden
-                  by an inherited constructor.  */
+             if (sfk == sfk_none
+                 || DECL_INHERITED_CTOR (fn)
+                 || TREE_CODE (fn) == TEMPLATE_DECL)
+               /* Member function templates and non-special member functions
+                  coexist if they are not equivalently constrained.  A member
+                  function is not hidden by an inherited constructor.  */
                continue;
 
              /* P0848: For special member functions, deleted, unsatisfied, or
diff --git a/gcc/testsuite/g++.dg/concepts/pr95181-2.C b/gcc/testsuite/g++.dg/concepts/pr95181-2.C
new file mode 100644 (file)
index 0000000..6d67350
--- /dev/null
@@ -0,0 +1,8 @@
+// { dg-do compile { target concepts } }
+
+template<bool B> struct g {
+  g() requires B && false;
+  g() requires B;
+};
+
+g<true> b; // error
diff --git a/gcc/testsuite/g++.dg/concepts/pr95181.C b/gcc/testsuite/g++.dg/concepts/pr95181.C
new file mode 100644 (file)
index 0000000..0185c86
--- /dev/null
@@ -0,0 +1,9 @@
+// PR c++/95181
+// { dg-do compile { target concepts } }
+
+template <typename> struct f {
+  template <typename T=int> f();
+  template <typename T=int> requires false f();
+};
+
+f<int> a;