[Lldb-commits] [lldb] [lldb] Add SetValueFromCString API to SyntheticFronend (PR #67309)

Alex Langford via lldb-commits lldb-commits at lists.llvm.org
Mon Sep 25 13:49:15 PDT 2023


================
@@ -34,7 +34,9 @@ class SyntheticChildrenFrontEnd {
 
 public:
   SyntheticChildrenFrontEnd(ValueObject &backend)
-      : m_backend(backend), m_valid(true) {}
+      : m_backend(backend), m_valid(true) {
+    backend.SetSyntheticFrontend(this);
+  }
----------------
bulbazord wrote:

Ignoring whether or not ValueObject should hold a pointer to a SyntheticFrontEnd, I'm concerned about the lifetime management of this solution. If a SyntheticChildrenFrontEnd is destroyed, the underlying `ValueObject` will be holding a dangling pointer.

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


More information about the lldb-commits mailing list