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



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to