friss added a comment.

In D77153#1954154 <https://reviews.llvm.org/D77153#1954154>, @labath wrote:

> It might be worth mentioning though that the title of the patch is somewhat 
> misleading -- I believe the `CXX` in `CXXFunctionSummaryFormat` refers to the 
> language the formatter is implemented in, not the language of the type 
> itself. So this patch affects all types whose formatters are implemented as 
> c++ functions, not all c++ pointers in the target program.


Wow, thanks for pointing this out. It should have clicked when I saw 
`ScriptSummaryFormat` besides it, but I was indeed completely confused. Let me 
rework this


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D77153/new/

https://reviews.llvm.org/D77153



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

Reply via email to