[Lldb-commits] [lldb] [lldb] [mostly NFC] Large WP foundation: WatchpointResources (PR #68845)

Alex Langford via lldb-commits lldb-commits at lists.llvm.org
Mon Oct 23 17:58:54 PDT 2023


================
@@ -491,14 +491,13 @@ static StopInfoSP GetStopInfoForHardwareBP(Thread &thread, Target *target,
                                            uint64_t exc_sub_sub_code) {
   // Try hardware watchpoint.
   if (target) {
+    // LWP_TODO: We need to find the WatchpointResource that matches
----------------
bulbazord wrote:

`LWP_TODO` doesn't mean much to a lot of contributors, maybe you can change this to something like `TODO(Large Watchpoint Support)`? Mostly to avoid acronyms that aren't written down anywhere else.

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


More information about the lldb-commits mailing list