[Lldb-commits] [PATCH] D45547: [Command] Implement `stats`

Jason Molenda via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Wed Apr 11 17:26:04 PDT 2018


jasonmolenda added a comment.

My two cents,

Why print the '### Start STATISTICS dump ###' header/footer when printing the results?  I don't think we demarcate result output like that anywhere else in lldb.  I don't think it adds anything for the user.

I would probably name the command statistics and have a built in "stats" alias.

Should statistics disable print the results, or should there be a 'statistics status' or 'statistics show' or 'statistics dump' command to show the statistics results accumulated so far?


https://reviews.llvm.org/D45547





More information about the lldb-commits mailing list