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