[Lldb-commits] [lldb] Support statistics dump summary only mode (PR #80745)

Greg Clayton via lldb-commits lldb-commits at lists.llvm.org
Mon Feb 5 20:52:03 PST 2024


================
@@ -130,10 +130,14 @@ struct ConstStringStats {
   ConstString::MemoryStats stats = ConstString::GetMemoryStats();
 };
 
+struct StatisticsOptions {
+  bool summary_only = false;
+};
+
 /// A class that represents statistics for a since lldb_private::Target.
 class TargetStats {
 public:
-  llvm::json::Value ToJSON(Target &target);
+  llvm::json::Value ToJSON(Target &target, bool summary_only = false);
----------------
clayborg wrote:

We should still pass in a "const StatisticsOptions options = StatisticsOptions()" instead of a boolean here. If we add more options in the future, we should avoid changing this API that way.

https://github.com/llvm/llvm-project/pull/80745


More information about the lldb-commits mailing list