re PR c++/48211 ([C++0x] Segment Fault When Compiling)

	PR c++/48211
	* name-lookup.h (cp_class_binding): Make base a pointer.
	* name-lookup.c (new_class_binding): Adjust.
	(poplevel_class): Adjust.

From-SVN: r174303

diff --git a/gcc/cp/ChangeLog b/gcc/cp/ChangeLog
index 8e9b25f93e91df774bfe6c49370640d397cb81ef..4835d8498eb34ade68addf6a30f99625e838b1ce 100644 (file)
--- a/gcc/cp/ChangeLog
+++ b/gcc/cp/ChangeLog
@@ -1,5 +1,10 @@
 2011-05-26  Jason Merrill  <jason@redhat.com>
 
+       PR c++/48211
+       * name-lookup.h (cp_class_binding): Make base a pointer.
+       * name-lookup.c (new_class_binding): Adjust.
+       (poplevel_class): Adjust.
+
        PR c++/48424
        * decl.c (grokparms): Function parameter packs don't need to
        go at the end.

diff --git a/gcc/cp/name-lookup.c b/gcc/cp/name-lookup.c
index bb6d4b934725d36cd2970ce08c1204556bee5fa6..935dd2a78abc45bedb2252e1ea9b4445a0b4211f 100644 (file)
--- a/gcc/cp/name-lookup.c
+++ b/gcc/cp/name-lookup.c
@@ -319,33 +319,11 @@ new_class_binding (tree name, tree value, tree type, cxx_scope *scope)
   cp_class_binding *cb;
   cxx_binding *binding;
 
-  if (VEC_length (cp_class_binding, scope->class_shadowed))
-    {
-      cp_class_binding *old_base;
-      old_base = VEC_index (cp_class_binding, scope->class_shadowed, 0);
-      if (VEC_reserve (cp_class_binding, gc, scope->class_shadowed, 1))
-       {
-         /* Fixup the current bindings, as they might have moved.  */
-         size_t i;
-
-         FOR_EACH_VEC_ELT (cp_class_binding, scope->class_shadowed, i, cb)
-           {
-             cxx_binding **b;
-             b = &IDENTIFIER_BINDING (cb->identifier);
-             while (*b != &old_base[i].base)
-               b = &((*b)->previous);
-             *b = &cb->base;
-           }
-       }
-      cb = VEC_quick_push (cp_class_binding, scope->class_shadowed, NULL);
-    }
-  else
     cb = VEC_safe_push (cp_class_binding, gc, scope->class_shadowed, NULL);
 
   cb->identifier = name;
-  binding = &cb->base;
+  cb->base = binding = cxx_binding_make (value, type);
   binding->scope = scope;
-  cxx_binding_init (binding, value, type);
   return binding;
 }
 
@@ -2725,7 +2703,10 @@ poplevel_class (void)
   if (level->class_shadowed)
     {
       FOR_EACH_VEC_ELT (cp_class_binding, level->class_shadowed, i, cb)
-       IDENTIFIER_BINDING (cb->identifier) = cb->base.previous;
+       {
+         IDENTIFIER_BINDING (cb->identifier) = cb->base->previous;
+         cxx_binding_free (cb->base);
+       }
       ggc_free (level->class_shadowed);
       level->class_shadowed = NULL;
     }

diff --git a/gcc/cp/name-lookup.h b/gcc/cp/name-lookup.h
index 4bf253f3680fbac3385f534554c6572ce78c9024..3309f0a836a745dc3a1c932efa2a5108fc5a0bcd 100644 (file)
--- a/gcc/cp/name-lookup.h
+++ b/gcc/cp/name-lookup.h
@@ -140,7 +140,7 @@ typedef enum tag_scope {
 } tag_scope;
 
 typedef struct GTY(()) cp_class_binding {
-  cxx_binding base;
+  cxx_binding *base;
   /* The bound name.  */
   tree identifier;
 } cp_class_binding;