[Lldb-commits] [PATCH] D40283: lldb: Use the DWARF linkage name when importing C++ methods

Jim Ingham via lldb-commits lldb-commits at lists.llvm.org
Wed Jan 10 10:35:41 PST 2018



> On Jan 9, 2018, at 9:31 PM, Nelson Elhage via Phabricator <reviews at reviews.llvm.org> wrote:
> 
> nelhage added a comment.
> 
> Hey -- Is there anything I can do to move this patch forward? Would it help to do something like only setting the attribute if the mangled name that *would* be generated doesn't match the one from the DWARF? Any such symbol is already broken for lldb, so such a patch feels perhaps safer and more conservative, at the cost of a bit of added complexity.

That sounds fine from a complexity standpoint, but I worry about the performance impact of doing more work than we already do on all mangled names we look at.

Jim

> 
> 
> https://reviews.llvm.org/D40283
> 
> 
> 



More information about the lldb-commits mailing list