On 10/1/2026 3:58 PM, Pierrick Bouvier wrote:
On 9/30/2026 10:51 AM, Brian Cain wrote:
Add the hexagon-sys.xml gdb feature describing the system and guest
registers.
Signed-off-by: Brian Cain <[email protected]>
---
configs/targets/hexagon-softmmu.mak | 2 +-
gdbstub/gdb-xml/hexagon-sys.xml | 157 ++++++++++++++++++++++++++++
2 files changed, 158 insertions(+), 1 deletion(-)
create mode 100644 gdbstub/gdb-xml/hexagon-sys.xml
diff --git a/configs/targets/hexagon-softmmu.mak
b/configs/targets/hexagon-softmmu.mak
index 6cbdc64be56..1ba6420d205 100644
--- a/configs/targets/hexagon-softmmu.mak
+++ b/configs/targets/hexagon-softmmu.mak
@@ -1,7 +1,7 @@
# Default configuration for hexagon-softmmu
TARGET_ARCH=hexagon
-TARGET_XML_FILES=hexagon-core.xml hexagon-hvx.xml
+TARGET_XML_FILES=hexagon-core.xml hexagon-hvx.xml hexagon-sys.xml
TARGET_LONG_BITS=32
TARGET_NOT_USING_LEGACY_LDST_PHYS_API=y
TARGET_NOT_USING_LEGACY_NATIVE_ENDIAN_API=y
diff --git a/gdbstub/gdb-xml/hexagon-sys.xml b/gdbstub/gdb-xml/hexagon-sys.xml
new file mode 100644
index 00000000000..434f11c0747
--- /dev/null
+++ b/gdbstub/gdb-xml/hexagon-sys.xml
@@ -0,0 +1,157 @@
+<?xml version="1.0"?>
+<!--
+ Copyright(c) 2023-2025 Qualcomm Innovation Center, Inc. All Rights Reserved.
+
+ SPDX-License-Identifier: GPL-2.0-or-later
+
+ Note: this file is intended to be use with LLDB, so it contains fields
+ that may be unknown to GDB. For more information on such fields, please
+ see:
+
https://github.com/llvm/llvm-project/blob/287aa6c4536408413b860e61fca0318a27214cf3/lldb/docs/lldb-gdb-remote.txt#L738-L860
+
https://github.com/llvm/llvm-project/blob/287aa6c4536408413b860e61fca0318a27214cf3/lldb/source/Plugins/Process/gdb-remote/ProcessGDBRemote.cpp#L4275-L4335
+-->
small nit - could we reference a specific llvm version (tag) instead of
a commit?
Ok - can do. New URLs below are what I'll use instead:
https://github.com/llvm/llvm-project/blob/llvmorg-16.0.0/lldb/docs/lldb-gdb-remote.txt#L738-L860
https://github.com/llvm/llvm-project/blob/llvmorg-16.0.0/lldb/source/Plugins/Process/gdb-remote/ProcessGDBRemote.cpp#L4056-L4116
For the rest,
Reviewed-by: Pierrick Bouvier <[email protected]>
Regards,
Pierrick
