Tested x86_64-pc-linux-gnu, applying to trunk.

-- 8< --

In this testcase, the tree walk to look for bare parameter packs was
confused by finding a type with no TREE_BINFO.  But it should be fine that
it's unset; we already checked for unexpanded packs at parse time.

I also tried doing the partial instantiation of the local class, which is
probably the long-term direction we want to go, but for stage 4 let's go
with this safer change.

        PR c++/109241

gcc/cp/ChangeLog:

        * pt.cc (find_parameter_packs_r): Handle null TREE_BINFO.

gcc/testsuite/ChangeLog:

        * g++.dg/cpp1y/lambda-generic-local-class2.C: New test.
---
 gcc/cp/pt.cc                                        | 12 ++++++++----
 .../g++.dg/cpp1y/lambda-generic-local-class2.C      | 13 +++++++++++++
 2 files changed, 21 insertions(+), 4 deletions(-)
 create mode 100644 gcc/testsuite/g++.dg/cpp1y/lambda-generic-local-class2.C

diff --git a/gcc/cp/pt.cc b/gcc/cp/pt.cc
index 90bcaa78701..40deedc9ba9 100644
--- a/gcc/cp/pt.cc
+++ b/gcc/cp/pt.cc
@@ -4069,10 +4069,14 @@ find_parameter_packs_r (tree *tp, int *walk_subtrees, 
void* data)
     case TAG_DEFN:
       t = TREE_TYPE (t);
       if (CLASS_TYPE_P (t))
-       /* Local class, need to look through the whole definition.  */
-       for (tree bb : BINFO_BASE_BINFOS (TYPE_BINFO (t)))
-         cp_walk_tree (&BINFO_TYPE (bb), &find_parameter_packs_r,
-                       ppd, ppd->visited);
+       {
+         /* Local class, need to look through the whole definition.
+            TYPE_BINFO might be unset for a partial instantiation.  */
+         if (TYPE_BINFO (t))
+           for (tree bb : BINFO_BASE_BINFOS (TYPE_BINFO (t)))
+             cp_walk_tree (&BINFO_TYPE (bb), &find_parameter_packs_r,
+                           ppd, ppd->visited);
+       }
       else
        /* Enum, look at the values.  */
        for (tree l = TYPE_VALUES (t); l; l = TREE_CHAIN (l))
diff --git a/gcc/testsuite/g++.dg/cpp1y/lambda-generic-local-class2.C 
b/gcc/testsuite/g++.dg/cpp1y/lambda-generic-local-class2.C
new file mode 100644
index 00000000000..83856de1f41
--- /dev/null
+++ b/gcc/testsuite/g++.dg/cpp1y/lambda-generic-local-class2.C
@@ -0,0 +1,13 @@
+// PR c++/109241
+// { dg-do compile { target c++14 } }
+// { dg-options "" } no pedantic
+
+void g() {
+  [](auto) {
+    [](auto) {
+      ({
+        struct A {};
+      });
+    };
+  }(1);
+}

base-commit: cd0c433e5faba9a18f64881cd761a53a530aa798
-- 
2.31.1

Reply via email to