I can try to take some time to benchmark, hopefully this weekend or so.

I do want to reiterate that this without this change debugging binaries
linked against libstdc++ is badly degraded, since recent libstdc++  uses
ABI tags on ~any methods that touch a std::string.

- Nelson

On Thu, Jan 11, 2018 at 7:18 AM Davide Italiano <dccitali...@gmail.com>
wrote:

> On Thu, Jan 11, 2018 at 2:01 AM, Pavel Labath via lldb-commits
> <lldb-commits@lists.llvm.org> wrote:
> > On 10 January 2018 at 22:51, Greg Clayton <clayb...@gmail.com> wrote:
> >> The right solution seems to be adding some sort of custom GNU ABI tag
> to the DWARF. I know that won't help with existing binaries, but it sounds
> too expensive to set the ASM name for everything.
> >>
> >
> > What makes you think it will be expensive? I don't know much about
> > clang internals, but I think something has to provide a mangled name
> > in any case. So, if we don't specify a mangled name, clang will have
> > to do the mangling itself. For all I know, setting the asm attribute
> > may actually speed things up, as clang could avoid doing some extra
> > work.
> >
> > I think it would be interesting to try to measure the
> > performance/memory footprint impact of just setting the asm attribute
> > everywhere.
> >
>
> Indeed. If somebody has the time to put into this I think it's
> something worth benchmarking before declaring victory (or defeat).
>
> Thanks,
>
> --
> Davide
>
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to