================
@@ -421,6 +421,17 @@ void Watchpoint::SetEnabled(bool enabled, bool notify) {
   }
   bool changed = enabled != m_enabled;
   m_enabled = enabled;
+  // Update the current value, if this is a user enabling
+  // a watchpoint that was disabled.
+  // (lldb will disable and re-enable watchpoints as we step over
+  // the accesses, those will have notify==false)
+  if (notify && changed && enabled && m_target.GetProcessSP()) {
----------------
JDevlieghere wrote:

Previously this didn't touch inferior memory, but now that we do, should we 
check `StateIsStoppedState(process_sp->GetState(), true)` here? 

https://github.com/llvm/llvm-project/pull/219333
_______________________________________________
lldb-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to