[Lldb-commits] [PATCH] D110804: Add a new command "target metrics".

[Greg Clayton via Phabricator via lldb-commits]

clayborg added a comment.

I would like to get a consensus on if we should move this functionality to "statistics" or not. The reasons that I didn't do it were:

- "statistics" as a top level method doesn't really do what I wanted here, which was "target statistics", or stats that are related only to the current target. I am not all that interested in overall stats over all targets
- I didn't want to have to make a human readable and JSON output
- the "statistics enable"/"statistics disable" didn't seem to make sense as they were coded in top of tree. Maybe enabling and disabling makes more sense when more info is gathered in hidden patches that have not been upstreamed?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D110804