jingham added a comment.

I've been holding off on actually promoting the lldbDataFormatters until there 
was some tests written for these formatters.  llvm data structures are 
constantly changing and so formatters in these files often break.  It shouldn't 
be hard to build a little test executable that links to llvm or clang and makes 
structures of these types and makes sure the formatted results are correct.

But even so, the idea of leaving .lldbinit files lying around in really common 
places like the root of the llvm-project source directory gives me the 
willies...  It's not so bad since by default we won't use this, but then the 
suggestion for using this is to turn on the target.load-cwd-lldbinit setting, 
and given how we see people cargo-cult .lldbinit files, I think this is likely 
to cause confusion.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D141219

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

Reply via email to