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
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to