[Lldb-commits] [lldb] r250180 - Fix cast in arm watchpoint handling code

Tamas Berghammer via lldb-commits lldb-commits at lists.llvm.org
Tue Oct 13 09:48:05 PDT 2015


Author: tberghammer
Date: Tue Oct 13 11:48:04 2015
New Revision: 250180

URL: http://llvm.org/viewvc/llvm-project?rev=250180&view=rev
Log:
Fix cast in arm watchpoint handling code

We had an incorrect sign extension when castion from a pointer to an
lldb::addr_t what broke the watchpoint hit detection on arm.

Modified:
    lldb/trunk/source/Plugins/Process/Linux/NativeProcessLinux.cpp

Modified: lldb/trunk/source/Plugins/Process/Linux/NativeProcessLinux.cpp
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/source/Plugins/Process/Linux/NativeProcessLinux.cpp?rev=250180&r1=250179&r2=250180&view=diff
==============================================================================
--- lldb/trunk/source/Plugins/Process/Linux/NativeProcessLinux.cpp (original)
+++ lldb/trunk/source/Plugins/Process/Linux/NativeProcessLinux.cpp Tue Oct 13 11:48:04 2015
@@ -1282,7 +1282,7 @@ NativeProcessLinux::MonitorSIGTRAP(const
     {
         // If a watchpoint was hit, report it
         uint32_t wp_index;
-        Error error = thread.GetRegisterContext()->GetWatchpointHitIndex(wp_index, (lldb::addr_t)info.si_addr);
+        Error error = thread.GetRegisterContext()->GetWatchpointHitIndex(wp_index, (uintptr_t)info.si_addr);
         if (error.Fail() && log)
             log->Printf("NativeProcessLinux::%s() "
                         "received error while checking for watchpoint hits, "




More information about the lldb-commits mailing list