[Lldb-commits] [lldb] ValueImpl::GetSP was calling GetError before acquiring the StopLocker. (PR #225976)

[Will Hawkins via lldb-commits]

================
@@ -480,8 +480,15 @@ class ValueObject {
   virtual bool GetDeclaration(Declaration &decl);
 
   // The functions below should NOT be modified by subclasses
+  /// Updates the ValueObject's value if needed, and then returns the
+  /// current error state.  This should only be called if the stop locker is
+  /// held so the process can't resume while this is in flight.
----------------
hawkinsw wrote:

```suggestion
  /// held so the process can't resume while this update/access is in flight.
```

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