On Thu, Sep 22, 2011 at 05:40:15PM -0700, Cary Coutant wrote:
> > Ping?
> >
> > http://gcc.gnu.org/ml/gcc-patches/2011-08/msg01626.html

It looks good to me, but I'd prefer if Ian with his demangler author
and libiberty maintainer hats looked at it too.

> > include/ChangeLog:
> >
> >        PR 40831
> >        * demangle.h (enum demangle_component_type): Add
> >        DEMANGLE_COMPONENT_CLONE.
> >
> > libiberty/ChangeLog:
> >
> >        PR 40831
> >        * cp-demangle.c (d_make_comp): Add new component type.
> >        (cplus_demangle_mangled_name): Check for clone suffixes.
> >        (d_parmlist): Don't error out if we see '.'.
> >        (d_clone_suffix): New function.
> >        (d_print_comp): Print info for clone suffixes.
> >        * testsuite/demangle-expected: Add new testcases.

Maybe the testcase should test also
_Z2f1IiEvT_S0_S0_._omp_fn.2
_Z3fooi._omp_cpyfn.6
as examples of OpenMP suffixing - the former is an example of name
used for outlined #pragma omp {parallel,task} region from a function
and the latter is for copy constructors needed for #pragma omp task.

        Jakub

Reply via email to