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

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

Reply via email to