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

[Davide Italiano via Phabricator via lldb-commits]

davide added a comment.

In https://reviews.llvm.org/D40283#983946, @aprantl wrote:

> In https://reviews.llvm.org/D40283#983908, @clayborg wrote:
>
> > I am fine with checking this. The only issue on my end is the extra memory that will be needed to store these often huge mangled names in every AST context for the 0.0001% of the time it is actually needed.
>
>
> Well since this is for libstdc++ only, should we make this a platform-specific behavior and, e.g., disable it on Darwin?


Could be, but is it really worth the complexity? I'd rather defer this if we find it being a performance problem.
Either way is fine to me, FWIW.


https://reviews.llvm.org/D40283