From: Richard Sandiford Date: Mon, 23 Nov 2020 09:06:59 +0000 (+0000) Subject: c++: Add missing verify_type_context call [PR97904] X-Git-Tag: upstream/12.2.0~11711 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=d3585f5d0df47ffa453f5fe436fdf588301e5314;p=platform%2Fupstream%2Fgcc.git c++: Add missing verify_type_context call [PR97904] When adding the verify_type_context target hook, I'd missed a site that needs to check an array element type. gcc/cp/ PR c++/97904 * pt.c (tsubst): Use verify_type_context to check the type of an array element. gcc/testsuite/ PR c++/97904 * g++.dg/ext/sve-sizeless-1.C: Add more template tests. * g++.dg/ext/sve-sizeless-2.C: Likewise. --- diff --git a/gcc/cp/pt.c b/gcc/cp/pt.c index b037786..fdd7f2d 100644 --- a/gcc/cp/pt.c +++ b/gcc/cp/pt.c @@ -15868,6 +15868,10 @@ tsubst (tree t, tree args, tsubst_flags_t complain, tree in_decl) return error_mark_node; } + if (!verify_type_context (input_location, TCTX_ARRAY_ELEMENT, type, + !(complain & tf_error))) + return error_mark_node; + r = build_cplus_array_type (type, domain); if (!valid_array_size_p (input_location, r, in_decl, diff --git a/gcc/testsuite/g++.dg/ext/sve-sizeless-1.C b/gcc/testsuite/g++.dg/ext/sve-sizeless-1.C index 7f82922..9f05ca5 100644 --- a/gcc/testsuite/g++.dg/ext/sve-sizeless-1.C +++ b/gcc/testsuite/g++.dg/ext/sve-sizeless-1.C @@ -72,10 +72,37 @@ template class templated_struct4; template struct templated_struct5 : T {}; // { dg-error {base type '[^']*' fails to be a struct or class type} } template class templated_struct5; +template struct templated_struct6 { T x[N]; }; // { dg-error {array elements cannot have SVE type '(__SVInt8_t|svint8_t)'} } +template class templated_struct6; + +template +struct templated_struct7 { + static const int size = sizeof (T); // { dg-error {SVE type '(__SVInt8_t|svint8_t)' does not have a fixed size} } +#if __cplusplus >= 201103L + static const int align = alignof (T); // { dg-error {SVE type '(__SVInt8_t|svint8_t)' does not have a defined alignment} "" { target c++11 } } +#endif + + void f1 (T (&)[2]); // { dg-error {array elements cannot have SVE type '(__SVInt8_t|svint8_t)'} } +#if __cplusplus >= 201103L + auto f2 () -> decltype (new T); // { dg-error {cannot allocate objects with SVE type '(__SVInt8_t|svint8_t)'} "" { target c++11 } } + auto f3 (T *a) -> decltype (delete a); // { dg-error {cannot delete objects with SVE type '(__SVInt8_t|svint8_t)'} "" { target c++11 } } +#else + void f2 () throw (T); // { dg-error {cannot throw or catch SVE type '(__SVInt8_t|svint8_t)'} "" { target c++98_only } } +#endif +}; +template class templated_struct7; + +template struct templated_struct8 { typedef int type; }; + +template +void sfinae_f1 (typename templated_struct8::type); +template +void sfinae_f1 (T &); + #if __cplusplus >= 201103L template using typedef_sizeless1 = svint8_t; template using typedef_sizeless1 = svint8_t; -template using array = T[2]; +template using array = T[2]; // { dg-error {array elements cannot have SVE type '(svint8_t|__SVInt8_t)'} "" { target c++11 } } #endif // Pointers to sizeless types. @@ -119,7 +146,7 @@ statements (int n) __alignof (ext_produce_sve_sc ()); // { dg-error {SVE type 'svint8_t' does not have a defined alignment} } #if __cplusplus >= 201103L - array foo = {}; // { dg-error {array elements cannot have SVE type '(svint8_t|__SVInt8_t)'} "" { target c++11 } } + array foo = {}; // { dg-message {required from here} "" { target c++11 } } #endif // Initialization. @@ -298,6 +325,8 @@ statements (int n) thrower2 (); #endif + sfinae_f1 (sve_sc1); + // Use in traits. Doesn't use static_assert so that tests work with // earlier -std=s. diff --git a/gcc/testsuite/g++.dg/ext/sve-sizeless-2.C b/gcc/testsuite/g++.dg/ext/sve-sizeless-2.C index 40b65d3..0b86d9e 100644 --- a/gcc/testsuite/g++.dg/ext/sve-sizeless-2.C +++ b/gcc/testsuite/g++.dg/ext/sve-sizeless-2.C @@ -72,10 +72,37 @@ template class templated_struct4; template struct templated_struct5 : T {}; // { dg-error {base type '[^']*' fails to be a struct or class type} } template class templated_struct5; +template struct templated_struct6 { T x[N]; }; // { dg-error {array elements cannot have SVE type '(__SVInt8_t|svint8_t)'} } +template class templated_struct6; + +template +struct templated_struct7 { + static const int size = sizeof (T); // { dg-error {SVE type '(__SVInt8_t|svint8_t)' does not have a fixed size} } +#if __cplusplus >= 201103L + static const int align = alignof (T); // { dg-error {SVE type '(__SVInt8_t|svint8_t)' does not have a defined alignment} "" { target c++11 } } +#endif + + void f1 (T (&)[2]); // { dg-error {array elements cannot have SVE type '(__SVInt8_t|svint8_t)'} } +#if __cplusplus >= 201103L + auto f2 () -> decltype (new T); // { dg-error {cannot allocate objects with SVE type '(__SVInt8_t|svint8_t)'} "" { target c++11 } } + auto f3 (T *a) -> decltype (delete a); // { dg-error {cannot delete objects with SVE type '(__SVInt8_t|svint8_t)'} "" { target c++11 } } +#else + void f2 () throw (T); // { dg-error {cannot throw or catch SVE type '(__SVInt8_t|svint8_t)'} "" { target c++98_only } } +#endif +}; +template class templated_struct7; + +template struct templated_struct8 { typedef int type; }; + +template +void sfinae_f1 (typename templated_struct8::type); +template +void sfinae_f1 (T &); + #if __cplusplus >= 201103L template using typedef_sizeless1 = svint8_t; template using typedef_sizeless1 = svint8_t; -template using array = T[2]; +template using array = T[2]; // { dg-error {array elements cannot have SVE type '(svint8_t|__SVInt8_t)'} "" { target c++11 } } #endif // Pointers to sizeless types. @@ -119,7 +146,7 @@ statements (int n) __alignof (ext_produce_sve_sc ()); // { dg-error {SVE type 'svint8_t' does not have a defined alignment} } #if __cplusplus >= 201103L - array foo = {}; // { dg-error {array elements cannot have SVE type '(svint8_t|__SVInt8_t)'} "" { target c++11 } } + array foo = {}; // { dg-message {required from here} "" { target c++11 } } #endif // Initialization. @@ -298,6 +325,8 @@ statements (int n) thrower2 (); #endif + sfinae_f1 (sve_sc1); + // Use in traits. Doesn't use static_assert so that tests work with // earlier -std=s.