[LLDB][MIPS] fix watchpoint searched on client side for same masked variables

[mohit.bhakkad]

Summary

On MIPS architecture, it is possible to use same hardware register for 2 different variables with same watch mask(address & ~7).
wp_addr passed by server is sometime of other same masked variable, so we always rely on wp_hit_addr to search the
correct watchpoint.

[brucem]

source/Plugins/Process/gdb-remote/ProcessGDBRemote.cpp

2108 ↗

(On Diff #36838)

Missing a space between if and (.

[clayborg]

Add space in between if and opening ( and this is good to go.

[mohit.bhakkad]

In D13548#262873, @clayborg wrote:

Add space in between if and opening ( and this is good to go.

Thanks, addressed it while commiting.

lldb/trunk/source/Plugins/Process/gdb-remote/ProcessGDBRemote.cpp

if (tid != LLDB_INVALID_THREAD_ID)
                         {
                             StringExtractor desc_extractor(description.c_str());
                             addr_t wp_addr = desc_extractor.GetU64(LLDB_INVALID_ADDRESS);
                             uint32_t wp_index = desc_extractor.GetU32(LLDB_INVALID_INDEX32);
                             addr_t wp_hit_addr = desc_extractor.GetU64(LLDB_INVALID_ADDRESS);
                             watch_id_t watch_id = LLDB_INVALID_WATCH_ID;
                             if (wp_addr != LLDB_INVALID_ADDRESS)
                             {
+                                if (wp_hit_addr != LLDB_INVALID_ADDRESS)
+                                    wp_addr = wp_hit_addr;
                                 WatchpointSP wp_sp = GetTarget().GetWatchpointList().FindByAddress(wp_addr);
                                 if (wp_sp)
                                 {
                                     wp_sp->SetHardwareIndex(wp_index);
                                     watch_id = wp_sp->GetID();
                                 }
                             }
                             if (watch_id == LLDB_INVALID_WATCH_ID)