RE: [PATCH 9/9] gdb, mi: Add -shadow-stack-list-frames command

["Schimpe, Christina" <christina.schimpe@intel.com>]

[-- Attachment #1: Type: text/plain, Size: 1304 bytes --]

> -----Original Message-----
> From: Tom Tromey <tom@tromey.com>
> Sent: Friday, October 3, 2025 10:18 PM
> To: Schimpe, Christina <christina.schimpe@intel.com>
> Cc: gdb-patches@sourceware.org
> Subject: Re: [PATCH 9/9] gdb, mi: Add -shadow-stack-list-frames command
> 
> >>>>> "Christina" == Christina Schimpe <christina.schimpe@intel.com> writes:
> 
> Christina> Add the mi command for the subcommand "backtrace shadow".
> Christina> Similar to the mi interface for the ordinary backtrace
> Christina> command, support low-frame and high-frame as command line
> parameters.
> 
> Christina> +      if (uiout->is_mi_like_p ())
> Christina> +	{
> Christina> +	  uiout->field_string
> Christina> +	    ("arch", (gdbarch_bfd_arch_info (gdbarch))-
> >printable_name);
> Christina> +	}
> 
> I was wondering if the arch field is really useful to clients.

The commit message of commit "MI: Print frame architecture when printing frames on an MI channel",
which added the field for -stack-list-frames, states:

"This is useful for MI clients that need to know the architecture in
order to perform further analysis, for example to use their own
disassembler to analyze machine code."

So I think we should have it for the mi command of "bt shadow" as well.

Regards,
Christina

[-- Attachment #2.1: Type: text/plain, Size: 329 bytes --]

Intel Deutschland GmbH
Registered Address: Dornacher Straße 1, 85622 Feldkirchen, Germany
Tel: +49 89 991 430, www.intel.de
Managing Directors: Harry Demas, Jeffrey Schneiderman, Yin Chong Sorrell
Chairperson of the Supervisory Board: Nicole Lau
Registered Seat: Munich
Commercial Register: Amtsgericht München HRB 186928

[-- Attachment #2.2: Type: text/html, Size: 357 bytes --]