[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