[Lldb-commits] [lldb] [LLDB][Data Formatters] Calculate average and total time for summary providers within lldb (PR #102708)

Greg Clayton via lldb-commits lldb-commits at lists.llvm.org
Wed Sep 4 13:23:59 PDT 2024


================
@@ -258,6 +258,14 @@ class TypeSummaryImpl {
 
   virtual std::string GetDescription() = 0;
 
+  /// Get the name of the Type Summary Provider, either a C++ class, a summary
+  /// string, or a script function name.
+  virtual std::string GetName() = 0;
+
+  /// Get the name of the kind of Summary Provider, either c++, summary string,
+  /// script or python.
+  virtual std::string GetSummaryKindName() = 0;
----------------
clayborg wrote:

Should this be a string or an enum? I would vote for an `lldb_internal` enum. That way plug-ins know what to choose form and don't make up random stuff. 

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


More information about the lldb-commits mailing list