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

[Greg Clayton via Phabricator via lldb-commits]

clayborg added a comment.

Ok, so how does this sound:

- This new command will replace the existing "statistics dump" command and emit JSON only by default
- The "statistics enable/disable" will stay in place and allow expensive metric gathering to be enabled/disabled if needed. If the gathering is really cheap for certain metrics, then nothing needs to happen for "statistics enable/disable"
- By default we will emit stats for the current target, and options can be added in the future to enable dumping for all targets and for debugger level stuff

Let me know if this sounds ok to everyone!?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D110804