[Lldb-commits] [PATCH] Report stopped by trace if none of the watchpoint was hit

Tamas Berghammer tberghammer at google.com
Tue Mar 17 07:31:05 PDT 2015


================
Comment at: source/Plugins/Process/Linux/NativeRegisterContextLinux_x86_64.h:49
@@ -48,3 +48,3 @@
         Error
-        IsWatchpointVacant(uint32_t wp_index);
+        IsWatchpointVacant(uint32_t wp_index) override;
 
----------------
chaoren wrote:
> Is this necessary?
Which part?

The override keyword is optional but gives us some more compile time check.

http://reviews.llvm.org/D8081

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the lldb-commits mailing list