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

Davide Italiano via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Wed Apr 11 18:02:01 PDT 2018

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 :)


More information about the lldb-commits mailing list