[Lldb-commits] [PATCH] D141219: Add a .lldbinit file to autoload LLVM/Clang data formatters

[Jim Ingham via Phabricator via lldb-commits]

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