[compiler-rt] [compiler-rt] Implements DumpAllRegisters for windows intel archs. (PR #108688)
David CARLIER via llvm-commits
llvm-commits at lists.llvm.org
Sat Oct 12 12:45:47 PDT 2024
https://github.com/devnexen updated https://github.com/llvm/llvm-project/pull/108688
>From fb4b9fb43bd47dd77fff84389bbcbb9fdef1f291 Mon Sep 17 00:00:00 2001
From: David Carlier <devnexen at gmail.com>
Date: Sat, 14 Sep 2024 08:35:53 +0100
Subject: [PATCH 1/2] [compiler-rt] Implements DumpAllRegisters for windows
intel archs.
---
.../lib/sanitizer_common/sanitizer_win.cpp | 40 ++++++++++++++++++-
1 file changed, 39 insertions(+), 1 deletion(-)
diff --git a/compiler-rt/lib/sanitizer_common/sanitizer_win.cpp b/compiler-rt/lib/sanitizer_common/sanitizer_win.cpp
index 8a80d54751364e..0d1c229b70604b 100644
--- a/compiler-rt/lib/sanitizer_common/sanitizer_win.cpp
+++ b/compiler-rt/lib/sanitizer_common/sanitizer_win.cpp
@@ -1039,7 +1039,45 @@ SignalContext::WriteFlag SignalContext::GetWriteFlag() const {
}
void SignalContext::DumpAllRegisters(void *context) {
- // FIXME: Implement this.
+ CONTEXT *ctx = (CONTEXT *)context;
+# if defined(__M_X64)
+ Report("Register values:\n");
+ Printf("rax = %llx ", ctx->Rax);
+ Printf("rbx = %llx ", ctx->Rbx);
+ Printf("rcx = %llx ", ctx->Rcx);
+ Printf("rdx = %llx ", ctx->Rdx);
+ Printf("\n");
+ Printf("rdi = %llx ", ctx->Rdi);
+ Printf("rsi = %llx ", ctx->Rsi);
+ Printf("rbp = %llx ", ctx->Rbp);
+ Printf("rsp = %llx ", ctx->Rsp);
+ Printf("\n");
+ Printf("r8 = %llx ", ctx->R8);
+ Printf("r9 = %llx ", ctx->R9);
+ Printf("r10 = %llx ", ctx->R10);
+ Printf("r11 = %llx ", ctx->R11);
+ Printf("\n");
+ Printf("r12 = %llx ", ctx->R12);
+ Printf("r13 = %llx ", ctx->R13);
+ Printf("r14 = %llx ", ctx->R14);
+ Printf("r15 = %llx ", ctx->R15);
+ Printf("\n");
+# elif defined(_M_IX86)
+ Report("Register values:\n");
+ Printf("eax = %lx ", ctx->Eax);
+ Printf("ebx = %lx ", ctx->Ebx);
+ Printf("ecx = %lx ", ctx->Ecx);
+ Printf("edx = %lx ", ctx->Edx);
+ Printf("\n");
+ Printf("edi = %lx ", ctx->Edi);
+ Printf("esi = %lx ", ctx->Esi);
+ Printf("ebp = %lx ", ctx->Ebp);
+ Printf("esp = %lx ", ctx->Esp);
+ Printf("\n");
+# else
+ // TODO
+ (void)ctx;
+# endif
}
int SignalContext::GetType() const {
>From bc1a1263ceb16681e328c311429686f13b30c686 Mon Sep 17 00:00:00 2001
From: David Carlier <devnexen at gmail.com>
Date: Sat, 12 Oct 2024 20:45:33 +0100
Subject: [PATCH 2/2] add tests
---
.../Windows/dump_registers_aarch64.cpp | 23 +++++++++++++++++++
.../TestCases/Windows/dump_registers_arm.cpp | 19 +++++++++++++++
.../TestCases/Windows/dump_registers_i386.cpp | 17 ++++++++++++++
.../Windows/dump_registers_x86_64.cpp | 19 +++++++++++++++
4 files changed, 78 insertions(+)
create mode 100644 compiler-rt/test/sanitizer_common/TestCases/Windows/dump_registers_aarch64.cpp
create mode 100644 compiler-rt/test/sanitizer_common/TestCases/Windows/dump_registers_arm.cpp
create mode 100644 compiler-rt/test/sanitizer_common/TestCases/Windows/dump_registers_i386.cpp
create mode 100644 compiler-rt/test/sanitizer_common/TestCases/Windows/dump_registers_x86_64.cpp
diff --git a/compiler-rt/test/sanitizer_common/TestCases/Windows/dump_registers_aarch64.cpp b/compiler-rt/test/sanitizer_common/TestCases/Windows/dump_registers_aarch64.cpp
new file mode 100644
index 00000000000000..d1015a4e369561
--- /dev/null
+++ b/compiler-rt/test/sanitizer_common/TestCases/Windows/dump_registers_aarch64.cpp
@@ -0,0 +1,23 @@
+// Check that sanitizer prints registers dump_registers on dump_registers=1
+// RUN: %clangxx %s -o %t
+// RUN: %env_tool_opts=dump_registers=0 not %run %t 2>&1 | FileCheck %s --check-prefix=CHECK-NODUMP
+// RUN: not %run %t 2>&1 | FileCheck %s --strict-whitespace --check-prefix=CHECK-DUMP
+//
+// REQUIRES: aarch64-target-arch && glibc
+
+#include <signal.h>
+
+int main() {
+ raise(SIGSEGV);
+ // CHECK-DUMP: Register values
+ // CHECK-DUMP-NEXT: x0 = {{0x[0-9a-f]+}} x1 = {{0x[0-9a-f]+}} x2 = {{0x[0-9a-f]+}} x3 = {{0x[0-9a-f]+}}
+ // CHECK-DUMP-NEXT: x4 = {{0x[0-9a-f]+}} x5 = {{0x[0-9a-f]+}} x6 = {{0x[0-9a-f]+}} x7 = {{0x[0-9a-f]+}}
+ // CHECK-DUMP-NEXT: x8 = {{0x[0-9a-f]+}} x9 = {{0x[0-9a-f]+}} x10 = {{0x[0-9a-f]+}} x11 = {{0x[0-9a-f]+}}
+ // CHECK-DUMP-NEXT:x12 = {{0x[0-9a-f]+}} x13 = {{0x[0-9a-f]+}} x14 = {{0x[0-9a-f]+}} x15 = {{0x[0-9a-f]+}}
+ // CHECK-DUMP-NEXT:x16 = {{0x[0-9a-f]+}} x17 = {{0x[0-9a-f]+}} x18 = {{0x[0-9a-f]+}} x19 = {{0x[0-9a-f]+}}
+ // CHECK-DUMP-NEXT:x20 = {{0x[0-9a-f]+}} x21 = {{0x[0-9a-f]+}} x22 = {{0x[0-9a-f]+}} x23 = {{0x[0-9a-f]+}}
+ // CHECK-DUMP-NEXT:x24 = {{0x[0-9a-f]+}} x25 = {{0x[0-9a-f]+}} x26 = {{0x[0-9a-f]+}} x27 = {{0x[0-9a-f]+}}
+ // CHECK-DUMP-NEXT:x28 = {{0x[0-9a-f]+}} fp = {{0x[0-9a-f]+}} lr = {{0x[0-9a-f]+}} sp = {{0x[0-9a-f]+}}
+ // CHECK-NODUMP-NOT: Register values
+ return 0;
+}
diff --git a/compiler-rt/test/sanitizer_common/TestCases/Windows/dump_registers_arm.cpp b/compiler-rt/test/sanitizer_common/TestCases/Windows/dump_registers_arm.cpp
new file mode 100644
index 00000000000000..e747f78188d32a
--- /dev/null
+++ b/compiler-rt/test/sanitizer_common/TestCases/Windows/dump_registers_arm.cpp
@@ -0,0 +1,19 @@
+// Check that sanitizer prints registers dump_registers on dump_registers=1
+// RUN: %clangxx %s -o %t
+// RUN: %env_tool_opts=dump_registers=0 not %run %t 2>&1 | FileCheck %s --check-prefix=CHECK-NODUMP
+// RUN: not %run %t 2>&1 | FileCheck %s --strict-whitespace --check-prefix=CHECK-DUMP
+//
+// REQUIRES: arm-target-arch && glibc
+
+#include <signal.h>
+
+int main() {
+ raise(SIGSEGV);
+ // CHECK-DUMP: Register values
+ // CHECK-DUMP-NEXT: r0 = {{0x[0-9a-f]+}} r1 = {{0x[0-9a-f]+}} r2 = {{0x[0-9a-f]+}} r3 = {{0x[0-9a-f]+}}
+ // CHECK-DUMP-NEXT: r4 = {{0x[0-9a-f]+}} r5 = {{0x[0-9a-f]+}} r6 = {{0x[0-9a-f]+}} r7 = {{0x[0-9a-f]+}}
+ // CHECK-DUMP-NEXT: r8 = {{0x[0-9a-f]+}} r9 = {{0x[0-9a-f]+}} r10 = {{0x[0-9a-f]+}} r11 = {{0x[0-9a-f]+}}
+ // CHECK-DUMP-NEXT:r12 = {{0x[0-9a-f]+}} sp = {{0x[0-9a-f]+}} lr = {{0x[0-9a-f]+}} pc = {{0x[0-9a-f]+}}
+ // CHECK-NODUMP-NOT: Register values
+ return 0;
+}
diff --git a/compiler-rt/test/sanitizer_common/TestCases/Windows/dump_registers_i386.cpp b/compiler-rt/test/sanitizer_common/TestCases/Windows/dump_registers_i386.cpp
new file mode 100644
index 00000000000000..76c7a901e34fdf
--- /dev/null
+++ b/compiler-rt/test/sanitizer_common/TestCases/Windows/dump_registers_i386.cpp
@@ -0,0 +1,17 @@
+// Check that sanitizer prints registers dump_registers on dump_registers=1
+// RUN: %clangxx %s -o %t
+// RUN: %env_tool_opts=dump_registers=0 not %run %t 2>&1 | FileCheck %s --check-prefixes=CHECK-NODUMP --strict-whitespace
+// RUN: not %run %t 2>&1 | FileCheck %s --check-prefixes=CHECK-DUMP --strict-whitespace
+//
+// REQUIRES: i386-pc-windows-msvc
+
+#include <windows.h>
+
+int main() {
+ RaiseException(EXCEPTION_ACCESS_VIOLATION, 0, 0, NULL);
+ // CHECK-DUMP: Register values
+ // CHECK-DUMP-NEXT: eax = {{0x[0-9a-f]+}} ebx = {{0x[0-9a-f]+}} ecx = {{0x[0-9a-f]+}} edx = {{0x[0-9a-f]+}}
+ // CHECK-DUMP-NEXT: edi = {{0x[0-9a-f]+}} esi = {{0x[0-9a-f]+}} ebp = {{0x[0-9a-f]+}} esp = {{0x[0-9a-f]+}}
+ // CHECK-NODUMP-NOT: Register values
+ return 0;
+}
diff --git a/compiler-rt/test/sanitizer_common/TestCases/Windows/dump_registers_x86_64.cpp b/compiler-rt/test/sanitizer_common/TestCases/Windows/dump_registers_x86_64.cpp
new file mode 100644
index 00000000000000..17c51aa27066e2
--- /dev/null
+++ b/compiler-rt/test/sanitizer_common/TestCases/Windows/dump_registers_x86_64.cpp
@@ -0,0 +1,19 @@
+// Check that sanitizer prints registers dump_registers on dump_registers=1
+// RUN: %clangxx %s -o %t
+// RUN: %env_tool_opts=dump_registers=0 not %run %t 2>&1 | FileCheck %s --check-prefixes=CHECK-NODUMP --strict-whitespace
+// RUN: not %run %t 2>&1 | FileCheck %s --check-prefixes=CHECK-DUMP --strict-whitespace
+//
+// REQUIRES: x86_64-pc-windows-msvc
+
+#include <windows.h>
+
+int main() {
+ RaiseException(EXCEPTION_ACCESS_VIOLATION, 0, 0, NULL);
+ // CHECK-DUMP: Register values
+ // CHECK-DUMP-NEXT: rax = {{0x[0-9a-f]+}} rbx = {{0x[0-9a-f]+}} rcx = {{0x[0-9a-f]+}} rdx = {{0x[0-9a-f]+}}
+ // CHECK-DUMP-NEXT: rdi = {{0x[0-9a-f]+}} rsi = {{0x[0-9a-f]+}} rbp = {{0x[0-9a-f]+}} rsp = {{0x[0-9a-f]+}}
+ // CHECK-DUMP-NEXT: r8 = {{0x[0-9a-f]+}} r9 = {{0x[0-9a-f]+}} r10 = {{0x[0-9a-f]+}} r11 = {{0x[0-9a-f]+}}
+ // CHECK-DUMP-NEXT: r12 = {{0x[0-9a-f]+}} r13 = {{0x[0-9a-f]+}} r14 = {{0x[0-9a-f]+}} r15 = {{0x[0-9a-f]+}}
+ // CHECK-NODUMP-NOT: Register values
+ return 0;
+}
More information about the llvm-commits
mailing list