[Lldb-commits] [PATCH] D110804: Add a new command "target metrics".
Pavel Labath via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Thu Sep 30 04:08:34 PDT 2021
labath added inline comments.
================
Comment at: lldb/include/lldb/Core/Module.h:940
+ /// ElapsedTime
+ double &GetSymtabParseTime() { return m_symtab_parse_time; }
+ double &GetSymtabNamesTime() { return m_symtab_names_time; }
----------------
teemperor wrote:
> Could the `double` and `uint64_t` here/below be a typedef to some unit? `typedef double Seconds`/`typedef uint64_t ByteCount` or something along those lines. Would make the unit we're expecting more obvious and I can easily find all the related code.
*cough* *cough* std::chrono *cough* *cough*
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D110804/new/
https://reviews.llvm.org/D110804
More information about the lldb-commits
mailing list