Fix review comments from David

Author: omjavaid

lldb/source/Plugins/Process/Windows/Common/NativeProcessWindows.cpp

@@ -502,13 +502,17 @@ NativeProcessWindows::OnDebugException(bool first_chance,
       if (FindSoftwareBreakpoint(exception_addr)) {
         LLDB_LOG(log, "Hit non-loader breakpoint at address {0:x}.",
                  exception_addr);
-
+        // The current PC is AFTER the BP opcode, on all architectures.
         reg_ctx.SetPC(reg_ctx.GetPC() - GetSoftwareBreakpointPCOffset());
         StopThread(thread_id, StopReason::eStopReasonBreakpoint);
         SetState(eStateStopped, true);
         return ExceptionResult::MaskException;
       } else {
         const std::vector<ULONG_PTR> &args = record.GetExceptionArguments();
+        // Check that the ExceptionInformation array of EXCEPTION_RECORD
+        // contains at least two elements: the first is a read-write flag
+        // indicating the type of data access operation (read or write) while
+        // the second contains the virtual address of the accessed data.
         if (args.size() >= 2) {
           uint32_t hw_id = LLDB_INVALID_INDEX32;
           reg_ctx.GetWatchpointHitIndex(hw_id, args[1]);

lldb/source/Plugins/Process/Windows/Common/NativeRegisterContextWindows_arm64.cpp

@@ -148,7 +148,6 @@ NativeRegisterContextWindows_arm64::NativeRegisterContextWindows_arm64(
   // breakpoints and watchpoints on Windows. The values set below are based
   // on tests conducted on Windows 11 with Snapdragon Elite X hardware.
   m_max_hwp_supported = 1;
-  m_max_hbp_supported = 0;
 }
 
 bool NativeRegisterContextWindows_arm64::IsGPR(uint32_t reg_index) const {