[Lldb-commits] [lldb] Make ValueObjectPrinter's handling of its ValueObject pointers more principled (NFC) (PR #81314)

[Greg Clayton via lldb-commits]

clayborg wrote:

None of what I suggested was mandatory. I was mainly trying to make sure things don't go away on us without us knowing about it. If we have crashes it is always hard to tell if simple heap corruption is the issue or if we are having lifetime issues. If you are not concerned about lifetime issues then you can ignore the `std::weak_ptr<T>` suggestion, but if you are, this would be a good way to tell as we can check `std::weak_ptr<T>::expired()`. 

Running a lldb with libgmalloc is a good way to watch for pointer lifetime issues and things being freed if someone ever has a repro case that they add to a bug.

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