On 03/13/2017 06:29 PM, Mark Wielaard wrote: > O, sorry. I should have let Nick known about the gdb regressions I found. > Besides this patch gdb needs the following one-liner fix: > > diff --git a/gdb/cp-name-parser.y b/gdb/cp-name-parser.y > index fd1e949..5278c05 100644 > --- a/gdb/cp-name-parser.y > +++ b/gdb/cp-name-parser.y > @@ -201,6 +201,7 @@ make_empty (enum demangle_component_type d_type) > { > struct demangle_component *ret = d_grab (); > ret->type = d_type; > + ret->d_printing = 0; > return ret; > }
Should gdb be memsetting instead to avoid having to know about libiberty's "internals"? Either version is pre-approved for GDB. Thanks, Pedro Alves