[Lldb-commits] [lldb] [lldb][Formatters] Do not recursively dereference pointer type when creating formatter candicates list. (PR #124048)

Pavel Labath via lldb-commits lldb-commits at lists.llvm.org
Thu Jan 23 00:08:03 PST 2025


================
@@ -222,14 +223,15 @@ void FormatManager::GetPossibleMatches(
 
   if (compiler_type.IsPointerType()) {
     CompilerType non_ptr_type = compiler_type.GetPointeeType();
-    GetPossibleMatches(valobj, non_ptr_type, use_dynamic, entries,
-                       current_flags.WithStrippedPointer());
+    if (dereference_ptr)
+      GetPossibleMatches(valobj, non_ptr_type, use_dynamic, entries,
+                        current_flags.WithStrippedPointer(), false);
----------------
labath wrote:

Are you sure we need the extra flag? Could we use `current_flags.stripped_pointer` as an indicator of whether we've removed a level of pointers?

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


More information about the lldb-commits mailing list