clayborg requested changes to this revision.
clayborg added a comment.
This revision now requires changes to proceed.
Just a few questions on why read watchpoints were being disabled in the few
SBValue::Watch() calls. Other than that it looks good. Is it possible to revert
the read watchpoint changes, or is it needed for SystemZ?
================
Comment at:
packages/Python/lldbsuite/test/functionalities/watchpoint/watchpoint_events/TestWatchpointEvents.py:62
@@ -61,3 +61,3 @@
error = lldb.SBError()
- local_watch = local_var.Watch(True, True, True, error)
+ local_watch = local_var.Watch(True, False, True, error)
if not error.Success():
----------------
I realize this test isn't actually using the watchpoint, it is just testing it,
but why is this change needed? Does SystemZ not support read watchpoints or
something?
================
Comment at:
packages/Python/lldbsuite/test/python_api/watchpoint/TestWatchpointIter.py:61
@@ -60,3 +60,3 @@
error = lldb.SBError();
- watchpoint = value.Watch(True, True, True, error)
+ watchpoint = value.Watch(True, False, True, error)
self.assertTrue(value and watchpoint,
----------------
This test just tests hitting writes to a variable, but why is this change
needed? Does SystemZ not support read watchpoints or something?
http://reviews.llvm.org/D18978
_______________________________________________
lldb-commits mailing list
[email protected]
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
