augusto2112 accepted this revision.
augusto2112 added a comment.
This revision is now accepted and ready to land.

I've bumped into this problem before, so really like this change. It's a bit 
sad that you'll only print the kind of data formatter once though, so someone 
reading the logs might have to jump back and forth to understand what kind of 
formatter they're dealing with. There's also `__PRETTY_FUNCTION__`, which shows 
the generic type parameters by default, but I think it's not a standard macro 
(although it's implemented by both clang and gcc). What do you think of having 
a macro like:

  #ifndef __PRETTY_FUNCTION__
     #define __PRETTY_FUNCTION__ __FUNCTION__
  #endif

And replacing `__FUNCTION__` with `__PRETTY_FUNCTION__`?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D154128

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

Reply via email to