[Lldb-commits] [PATCH] D137000: [lldb] Make callback-based formatter matching available from the CLI.

Jim Ingham via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Tue Nov 8 14:01:29 PST 2022


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

It is a bit odd to follow the what the help summary says is a valid option set and then get an error.   But this is a corner case, and since you say explicitly that you can't provide both that's probably good enough.  Would you mind changing "Incompatible with" to: "Cannot be specified at the same time as".  Incompatible isn't precise - does it mean you can't use them both, or that if you do something weird will happen?

Other than that, this is fine.  Thanks for working on this.


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

https://reviews.llvm.org/D137000



More information about the lldb-commits mailing list