c++: visibility wrt concept-id as targ [PR115283]

Like with alias templates, it seems we don't maintain visibility flags
for concepts either, so min_vis_expr_r should ignore them for now.
Otherwise after r14-6789 we may incorrectly give a function template that
uses a concept-id in its signature internal linkage.

	PR c++/115283

gcc/cp/ChangeLog:

	* decl2.cc (min_vis_expr_r) <case TEMPLATE_DECL>: Ignore
	concepts.

gcc/testsuite/ChangeLog:

	* g++.dg/template/linkage5.C: New test.

Reviewed-by: Jason Merrill <jason@redhat.com>
This commit is contained in:
Patrick Palka
2024-06-12 20:05:05 -04:00
parent 5288935d30
commit b1fe718cbe
2 changed files with 17 additions and 2 deletions

View File

@@ -2723,9 +2723,10 @@ min_vis_expr_r (tree *tp, int */*walk_subtrees*/, void *data)
break;
case TEMPLATE_DECL:
if (DECL_ALIAS_TEMPLATE_P (t))
if (DECL_ALIAS_TEMPLATE_P (t) || standard_concept_p (t))
/* FIXME: We don't maintain TREE_PUBLIC / DECL_VISIBILITY for
alias templates so we can't trust it here (PR107906). */
alias templates so we can't trust it here (PR107906). Ditto
for concepts. */
break;
t = DECL_TEMPLATE_RESULT (t);
/* Fall through. */

View File

@@ -0,0 +1,14 @@
// PR c++/115283
// { dg-final { scan-assembler "(weak|glob)\[^\n\]*_Z1fIiEv1AIX1CIT_EEE" } }
// { dg-do compile { target c++20 } }
template<class T>
concept C = true;
template<bool B>
struct A { };
template<class T>
void f(A<C<T>>) { }
template void f<int>(A<true>);