diff options
author | Marek Polacek <polacek@redhat.com> | 2020-06-23 18:07:34 -0400 |
---|---|---|
committer | Marek Polacek <polacek@redhat.com> | 2020-06-29 10:59:40 -0400 |
commit | b1005f553d3543bb56dc6b9b34ee35455d697ca4 (patch) | |
tree | ddc4fa81ca1981181cb8c136953fe8abd262e86d /gcc | |
parent | e6cc67f6616c96f3e18a434e0c74ba2f3818cb6d (diff) | |
download | gcc-b1005f553d3543bb56dc6b9b34ee35455d697ca4.zip gcc-b1005f553d3543bb56dc6b9b34ee35455d697ca4.tar.gz gcc-b1005f553d3543bb56dc6b9b34ee35455d697ca4.tar.bz2 |
c++: Fix CTAD for aggregates in template [PR95568]
95568 complains that CTAD for aggregates doesn't work within
requires-clause and it turned out that it doesn't work when we try
the deduction in a template. The reason is that maybe_aggr_guide
creates a guide that can look like this
template<class T> X(decltype (X<T>::x))-> X<T>
where the parameter is a decltype, which is a non-deduced context. So
the subsequent build_new_function_call fails because unify_one_argument
can't deduce anything from it ([temp.deduct.type]: "If a template
parameter is used only in non-deduced contexts and is not explicitly
specified, template argument deduction fails.")
Those decltypes come from finish_decltype_type. We can just use
TREE_TYPE instead. I pondered using unlowered_expr_type, but that
didn't make any difference for the FIELD_DECLs I saw in
class-deduction-aggr6.C.
gcc/cp/ChangeLog:
PR c++/95568
* pt.c (collect_ctor_idx_types): Use TREE_TYPE.
gcc/testsuite/ChangeLog:
PR c++/95568
* g++.dg/cpp2a/class-deduction-aggr5.C: New test.
* g++.dg/cpp2a/class-deduction-aggr6.C: New test.
Diffstat (limited to 'gcc')
-rw-r--r-- | gcc/cp/pt.c | 2 | ||||
-rw-r--r-- | gcc/testsuite/g++.dg/cpp2a/class-deduction-aggr5.C | 20 | ||||
-rw-r--r-- | gcc/testsuite/g++.dg/cpp2a/class-deduction-aggr6.C | 35 |
3 files changed, 56 insertions, 1 deletions
diff --git a/gcc/cp/pt.c b/gcc/cp/pt.c index 53a64c3..618bf68 100644 --- a/gcc/cp/pt.c +++ b/gcc/cp/pt.c @@ -28329,7 +28329,7 @@ collect_ctor_idx_types (tree ctor, tree list, tree elt = NULL_TREE) tree idx, val; unsigned i; FOR_EACH_CONSTRUCTOR_ELT (v, i, idx, val) { - tree ftype = elt ? elt : finish_decltype_type (idx, true, tf_none); + tree ftype = elt ? elt : TREE_TYPE (idx); if (BRACE_ENCLOSED_INITIALIZER_P (val) && CONSTRUCTOR_NELTS (val) /* As in reshape_init_r, a non-aggregate or array-of-dependent-bound diff --git a/gcc/testsuite/g++.dg/cpp2a/class-deduction-aggr5.C b/gcc/testsuite/g++.dg/cpp2a/class-deduction-aggr5.C new file mode 100644 index 0000000..01253f4 --- /dev/null +++ b/gcc/testsuite/g++.dg/cpp2a/class-deduction-aggr5.C @@ -0,0 +1,20 @@ +// PR c++/95568 +// { dg-do compile { target c++20 } } + +template<typename T> struct X { T x; }; +template<typename T, typename U> struct X2 { T x; U y; }; +template<typename T> concept Y = requires { X{0}; }; + +template<typename T> +void g() +{ + X{0}; + X2{1, 2.2}; + Y auto y = X{1}; +} + +void +fn () +{ + g<int>(); +} diff --git a/gcc/testsuite/g++.dg/cpp2a/class-deduction-aggr6.C b/gcc/testsuite/g++.dg/cpp2a/class-deduction-aggr6.C new file mode 100644 index 0000000..95d7c5e --- /dev/null +++ b/gcc/testsuite/g++.dg/cpp2a/class-deduction-aggr6.C @@ -0,0 +1,35 @@ +// PR c++/95568 +// { dg-do compile { target c++20 } } +// CTAD with aggregates containing bit-fields. + +template<class, class> struct same_type; +template<class T> struct same_type<T, T> {}; + +enum E { e }; +enum class F { f }; + +template<typename T> +struct X { + T a : 5; +}; + +template<typename> +void g() +{ + auto x = X{ 0 }; + same_type<decltype(x.a), int>(); + auto x2 = X{ E::e }; + same_type<decltype(x2.a), E>(); + auto x3 = X{ false }; + same_type<decltype(x3.a), bool>(); + auto x4 = X{ 0u }; + same_type<decltype(x4.a), unsigned int>(); + auto x5 = X{ F::f }; + same_type<decltype(x5.a), F>(); +} + +void +fn () +{ + g<int>(); +} |