[Lldb-commits] [PATCH] D39967: Disable breakpoints before writing memory and re-enable after.
Tatyana Krasnukha via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Thu Feb 22 15:38:06 PST 2018
tatyana-krasnukha added inline comments.
Comment at: source/Target/Process.cpp:2437
+ for (auto &bp_site : enabled_bp_sites_in_range)
> If disabling any breakpoint fails, the status's error string will be "failed to re-enable one or more breakpoints". This is kind of confusing, imo.
Didn't found anything better than use word "update" here. Introducing additional flag looks ugly...
I hope, that this information is enough when user tries to write memory. More detailed description can be found in log.
More information about the lldb-commits