public inbox for gcc-cvs@sourceware.org
help / color / mirror / Atom feed
* [gcc r13-968] c++: ICE with template NEW_EXPR [PR105803]
@ 2022-06-02 20:01 Marek Polacek
0 siblings, 0 replies; only message in thread
From: Marek Polacek @ 2022-06-02 20:01 UTC (permalink / raw)
To: gcc-cvs
https://gcc.gnu.org/g:7b98910406b5000a6429c188b0c6cc14e3140637
commit r13-968-g7b98910406b5000a6429c188b0c6cc14e3140637
Author: Marek Polacek <polacek@redhat.com>
Date: Wed Jun 1 18:21:36 2022 -0400
c++: ICE with template NEW_EXPR [PR105803]
Here we ICE because value_dependent_expression_p gets a NEW_EXPR
whose operand is a type, and we go to the default case which just
calls v_d_e_p on each operand of the NEW_EXPR. Since one of them
is a type, we crash on the new assert in t_d_e_p.
t_d_e_p has code to handle {,VEC_}NEW_EXPR, which at this point
was already performed, so I think we can handle these two codes
specifically and skip the second operand, which is always going
to be a type.
PR c++/105803
gcc/cp/ChangeLog:
* pt.cc (value_dependent_expression_p): Handle {,VEC_}NEW_EXPR
in the switch.
gcc/testsuite/ChangeLog:
* g++.dg/template/new13.C: New test.
Diff:
---
gcc/cp/pt.cc | 8 ++++++++
gcc/testsuite/g++.dg/template/new13.C | 11 +++++++++++
2 files changed, 19 insertions(+)
diff --git a/gcc/cp/pt.cc b/gcc/cp/pt.cc
index 6de8e496859..836861e1039 100644
--- a/gcc/cp/pt.cc
+++ b/gcc/cp/pt.cc
@@ -27643,6 +27643,14 @@ value_dependent_expression_p (tree expression)
under instantiate_non_dependent_expr; it can't be constant. */
return true;
+ case NEW_EXPR:
+ case VEC_NEW_EXPR:
+ /* The second operand is a type, which type_dependent_expression_p
+ (and therefore value_dependent_expression_p) doesn't want to see. */
+ return (value_dependent_expression_p (TREE_OPERAND (expression, 0))
+ || value_dependent_expression_p (TREE_OPERAND (expression, 2))
+ || value_dependent_expression_p (TREE_OPERAND (expression, 3)));
+
default:
/* A constant expression is value-dependent if any subexpression is
value-dependent. */
diff --git a/gcc/testsuite/g++.dg/template/new13.C b/gcc/testsuite/g++.dg/template/new13.C
new file mode 100644
index 00000000000..17a19e40289
--- /dev/null
+++ b/gcc/testsuite/g++.dg/template/new13.C
@@ -0,0 +1,11 @@
+// PR c++/105803
+// { dg-do compile }
+// { dg-additional-options "-fchecking=2" }
+
+namespace std {
+template <typename> class shared_ptr {};
+}
+struct S {};
+template <int> void build_matrices() {
+ std::shared_ptr<S>(new S);
+}
^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2022-06-02 20:01 UTC | newest]
Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-06-02 20:01 [gcc r13-968] c++: ICE with template NEW_EXPR [PR105803] Marek Polacek
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).