tschuett added inline comments.
================
Comment at: lldb/source/Target/Statistics.cpp:266
+ auto stats = ts->ReportStatistics();
+ if (stats.hasValue()) {
+ module_stat_obj->try_emplace("TypeSystemInfo", stats.getValue());
----------------
clayborg wrote:
> Remove {} for single line if statement per llvm coding guidelines
I am bit confused. If `auto stats` is an `optional`. Please remove
`.hasValue()` and replace `stats.getValue` by `*stats`.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D137191/new/
https://reviews.llvm.org/D137191
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
