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

[Davide Italiano via Phabricator via lldb-commits]

davide added a comment.

In https://reviews.llvm.org/D45547#1065054, @jasonmolenda wrote:

> Ah, no you couldn't set up a command alias like that.  Still, if the full name was statistics, you could type 'stat' and it would match.  'stats' wouldn't, though.
>
> I do think decoupling the disabling action from the dumping action would be an improvement.  You may want to dump the current statistics multiple times during a sequence of commands, without disabling / re-enabling every time.  And dropping the header/footer from the results.


OK! I don't mind adding another `dump` subcommand. I think it makes sense. 
I can also disable the footer/header, I largely got that format from `opt -time-passes`. I don't feel strongly about it, so I'll let everybody else decide :)


https://reviews.llvm.org/D45547