github-actions[bot] wrote: <!--LLVM CODE FORMAT COMMENT: {clang-format}-->
:warning: C/C++ code formatter, clang-format found issues in your code. :warning: <details> <summary> You can test this locally with the following command: </summary> ``````````bash git-clang-format --diff 5537ae87b3a87b3abeb4e6983cecd9b103648243 1c29d30f0b9eb47da63c35f775ad69e1ef40c4de --extensions cpp,h -- lldb/source/Plugins/Process/Windows/Common/NativeProcessWindows.cpp lldb/source/Plugins/Process/Windows/Common/NativeRegisterContextWindows.cpp lldb/source/Plugins/Process/Windows/Common/NativeRegisterContextWindows.h lldb/source/Plugins/Process/Windows/Common/NativeRegisterContextWindows_WoW64.cpp lldb/source/Plugins/Process/Windows/Common/NativeRegisterContextWindows_arm.cpp lldb/source/Plugins/Process/Windows/Common/NativeRegisterContextWindows_arm64.cpp lldb/source/Plugins/Process/Windows/Common/NativeRegisterContextWindows_arm64.h lldb/source/Plugins/Process/Windows/Common/NativeRegisterContextWindows_i386.cpp lldb/source/Plugins/Process/Windows/Common/NativeRegisterContextWindows_x86_64.cpp lldb/source/Plugins/Process/Windows/Common/NativeThreadWindows.cpp `````````` </details> <details> <summary> View the diff from clang-format here. </summary> ``````````diff diff --git a/lldb/source/Plugins/Process/Windows/Common/NativeProcessWindows.cpp b/lldb/source/Plugins/Process/Windows/Common/NativeProcessWindows.cpp index 886df987dc..87a811123d 100644 --- a/lldb/source/Plugins/Process/Windows/Common/NativeProcessWindows.cpp +++ b/lldb/source/Plugins/Process/Windows/Common/NativeProcessWindows.cpp @@ -531,7 +531,7 @@ NativeProcessWindows::OnDebugException(bool first_chance, return ExceptionResult::MaskException; } } - } + } } if (!initial_stop) { diff --git a/lldb/source/Plugins/Process/Windows/Common/NativeRegisterContextWindows.cpp b/lldb/source/Plugins/Process/Windows/Common/NativeRegisterContextWindows.cpp index 95be1183ab..effe6df36b 100644 --- a/lldb/source/Plugins/Process/Windows/Common/NativeRegisterContextWindows.cpp +++ b/lldb/source/Plugins/Process/Windows/Common/NativeRegisterContextWindows.cpp @@ -18,7 +18,6 @@ using namespace lldb; using namespace lldb_private; - lldb::thread_t NativeRegisterContextWindows::GetThreadHandle() const { auto wthread = static_cast<NativeThreadWindows *>(&m_thread); return wthread->GetHostThread().GetNativeThread().GetSystemHandle(); diff --git a/lldb/source/Plugins/Process/Windows/Common/NativeRegisterContextWindows.h b/lldb/source/Plugins/Process/Windows/Common/NativeRegisterContextWindows.h index 66ced71d27..0f9e37dbd9 100644 --- a/lldb/source/Plugins/Process/Windows/Common/NativeRegisterContextWindows.h +++ b/lldb/source/Plugins/Process/Windows/Common/NativeRegisterContextWindows.h @@ -17,9 +17,9 @@ namespace lldb_private { class NativeThreadWindows; -class NativeRegisterContextWindows : public virtual NativeRegisterContextRegisterInfo { +class NativeRegisterContextWindows + : public virtual NativeRegisterContextRegisterInfo { public: - static std::unique_ptr<NativeRegisterContextWindows> CreateHostNativeRegisterContextWindows(const ArchSpec &target_arch, NativeThreadProtocol &native_thread); diff --git a/lldb/source/Plugins/Process/Windows/Common/NativeRegisterContextWindows_WoW64.cpp b/lldb/source/Plugins/Process/Windows/Common/NativeRegisterContextWindows_WoW64.cpp index 1ba89fbf32..069c327ee2 100644 --- a/lldb/source/Plugins/Process/Windows/Common/NativeRegisterContextWindows_WoW64.cpp +++ b/lldb/source/Plugins/Process/Windows/Common/NativeRegisterContextWindows_WoW64.cpp @@ -88,8 +88,8 @@ static Status SetWoW64ThreadContextHelper(lldb::thread_t thread_handle, NativeRegisterContextWindows_WoW64::NativeRegisterContextWindows_WoW64( const ArchSpec &target_arch, NativeThreadProtocol &native_thread) - : NativeRegisterContextRegisterInfo(native_thread, - CreateRegisterInfoInterface(target_arch)) {} + : NativeRegisterContextRegisterInfo( + native_thread, CreateRegisterInfoInterface(target_arch)) {} bool NativeRegisterContextWindows_WoW64::IsGPR(uint32_t reg_index) const { return (reg_index >= k_first_gpr_i386 && reg_index < k_first_alias_i386); diff --git a/lldb/source/Plugins/Process/Windows/Common/NativeRegisterContextWindows_arm.cpp b/lldb/source/Plugins/Process/Windows/Common/NativeRegisterContextWindows_arm.cpp index 470da1afc8..fd8a0c05c1 100644 --- a/lldb/source/Plugins/Process/Windows/Common/NativeRegisterContextWindows_arm.cpp +++ b/lldb/source/Plugins/Process/Windows/Common/NativeRegisterContextWindows_arm.cpp @@ -128,8 +128,8 @@ NativeRegisterContextWindows::CreateHostNativeRegisterContextWindows( NativeRegisterContextWindows_arm::NativeRegisterContextWindows_arm( const ArchSpec &target_arch, NativeThreadProtocol &native_thread) - : NativeRegisterContextRegisterInfo(native_thread, - CreateRegisterInfoInterface(target_arch)) {} + : NativeRegisterContextRegisterInfo( + native_thread, CreateRegisterInfoInterface(target_arch)) {} bool NativeRegisterContextWindows_arm::IsGPR(uint32_t reg_index) const { return (reg_index >= k_first_gpr_arm && reg_index <= k_last_gpr_arm); diff --git a/lldb/source/Plugins/Process/Windows/Common/NativeRegisterContextWindows_arm64.cpp b/lldb/source/Plugins/Process/Windows/Common/NativeRegisterContextWindows_arm64.cpp index ae09ebffa5..547ddcc403 100644 --- a/lldb/source/Plugins/Process/Windows/Common/NativeRegisterContextWindows_arm64.cpp +++ b/lldb/source/Plugins/Process/Windows/Common/NativeRegisterContextWindows_arm64.cpp @@ -142,8 +142,8 @@ NativeRegisterContextWindows::CreateHostNativeRegisterContextWindows( NativeRegisterContextWindows_arm64::NativeRegisterContextWindows_arm64( const ArchSpec &target_arch, NativeThreadProtocol &native_thread) - : NativeRegisterContextRegisterInfo(native_thread, - CreateRegisterInfoInterface(target_arch)) { + : NativeRegisterContextRegisterInfo( + native_thread, CreateRegisterInfoInterface(target_arch)) { // Currently, there is no API to query the maximum supported hardware // breakpoints and watchpoints on Windows. The values set below are based // on tests conducted on Windows 11 with Snapdragon Elite X hardware. @@ -716,8 +716,8 @@ Status NativeRegisterContextWindows_arm64::WriteAllRegisterValues( llvm::Error NativeRegisterContextWindows_arm64::ReadHardwareDebugInfo() { ::CONTEXT tls_context; - Status error = - GetThreadContextHelper(GetThreadHandle(), &tls_context, CONTEXT_DEBUG_REGISTERS); + Status error = GetThreadContextHelper(GetThreadHandle(), &tls_context, + CONTEXT_DEBUG_REGISTERS); if (error.Fail()) return error.ToError(); @@ -736,8 +736,8 @@ llvm::Error NativeRegisterContextWindows_arm64::ReadHardwareDebugInfo() { llvm::Error NativeRegisterContextWindows_arm64::WriteHardwareDebugRegs(DREGType hwbType) { ::CONTEXT tls_context; - Status error = - GetThreadContextHelper(GetThreadHandle(), &tls_context, CONTEXT_DEBUG_REGISTERS); + Status error = GetThreadContextHelper(GetThreadHandle(), &tls_context, + CONTEXT_DEBUG_REGISTERS); if (error.Fail()) return error.ToError(); diff --git a/lldb/source/Plugins/Process/Windows/Common/NativeRegisterContextWindows_arm64.h b/lldb/source/Plugins/Process/Windows/Common/NativeRegisterContextWindows_arm64.h index 164d46143d..e73a6af4cb 100644 --- a/lldb/source/Plugins/Process/Windows/Common/NativeRegisterContextWindows_arm64.h +++ b/lldb/source/Plugins/Process/Windows/Common/NativeRegisterContextWindows_arm64.h @@ -10,9 +10,9 @@ #ifndef liblldb_NativeRegisterContextWindows_arm64_h_ #define liblldb_NativeRegisterContextWindows_arm64_h_ -#include "Plugins/Process/Utility/lldb-arm64-register-enums.h" #include "Plugins/Process/Utility/NativeRegisterContextDBReg_arm64.h" #include "Plugins/Process/Utility/RegisterInfoPOSIX_arm64.h" +#include "Plugins/Process/Utility/lldb-arm64-register-enums.h" #include "NativeRegisterContextWindows.h" @@ -20,8 +20,9 @@ namespace lldb_private { class NativeThreadWindows; -class NativeRegisterContextWindows_arm64 : public NativeRegisterContextWindows, -public NativeRegisterContextDBReg_arm64 { +class NativeRegisterContextWindows_arm64 + : public NativeRegisterContextWindows, + public NativeRegisterContextDBReg_arm64 { public: NativeRegisterContextWindows_arm64(const ArchSpec &target_arch, NativeThreadProtocol &native_thread); diff --git a/lldb/source/Plugins/Process/Windows/Common/NativeRegisterContextWindows_i386.cpp b/lldb/source/Plugins/Process/Windows/Common/NativeRegisterContextWindows_i386.cpp index 71e3616d1a..7c13759df4 100644 --- a/lldb/source/Plugins/Process/Windows/Common/NativeRegisterContextWindows_i386.cpp +++ b/lldb/source/Plugins/Process/Windows/Common/NativeRegisterContextWindows_i386.cpp @@ -92,8 +92,8 @@ NativeRegisterContextWindows::CreateHostNativeRegisterContextWindows( NativeRegisterContextWindows_i386::NativeRegisterContextWindows_i386( const ArchSpec &target_arch, NativeThreadProtocol &native_thread) - : NativeRegisterContextRegisterInfo(native_thread, - CreateRegisterInfoInterface(target_arch)) {} + : NativeRegisterContextRegisterInfo( + native_thread, CreateRegisterInfoInterface(target_arch)) {} bool NativeRegisterContextWindows_i386::IsGPR(uint32_t reg_index) const { return (reg_index < k_first_alias_i386); diff --git a/lldb/source/Plugins/Process/Windows/Common/NativeRegisterContextWindows_x86_64.cpp b/lldb/source/Plugins/Process/Windows/Common/NativeRegisterContextWindows_x86_64.cpp index 3709390d95..c54c31e48e 100644 --- a/lldb/source/Plugins/Process/Windows/Common/NativeRegisterContextWindows_x86_64.cpp +++ b/lldb/source/Plugins/Process/Windows/Common/NativeRegisterContextWindows_x86_64.cpp @@ -110,8 +110,8 @@ NativeRegisterContextWindows::CreateHostNativeRegisterContextWindows( NativeRegisterContextWindows_x86_64::NativeRegisterContextWindows_x86_64( const ArchSpec &target_arch, NativeThreadProtocol &native_thread) - : NativeRegisterContextRegisterInfo(native_thread, - CreateRegisterInfoInterface(target_arch)) {} + : NativeRegisterContextRegisterInfo( + native_thread, CreateRegisterInfoInterface(target_arch)) {} bool NativeRegisterContextWindows_x86_64::IsGPR(uint32_t reg_index) const { return (reg_index >= k_first_gpr_x86_64 && reg_index < k_first_alias_x86_64); diff --git a/lldb/source/Plugins/Process/Windows/Common/NativeThreadWindows.cpp b/lldb/source/Plugins/Process/Windows/Common/NativeThreadWindows.cpp index cd5cd45410..cade9c09ff 100644 --- a/lldb/source/Plugins/Process/Windows/Common/NativeThreadWindows.cpp +++ b/lldb/source/Plugins/Process/Windows/Common/NativeThreadWindows.cpp @@ -192,7 +192,7 @@ Status NativeThreadWindows::SetHardwareBreakpoint(lldb::addr_t addr, return Status::FromErrorString("Setting hardware breakpoint failed."); m_hw_breakpoint_index_map.insert({addr, bp_index}); - + return Status(); #else return Status::FromErrorString("unimplemented."); `````````` </details> https://github.com/llvm/llvm-project/pull/108072 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits