Re: [PATCH v4 10/11] gdb, gdbarch: Introduce gdbarch method to get the shadow stack pointer.

Tom Tromey
2025-06-17T18:16:50Z

>>>>> "Christina" == Christina Schimpe <christina.schimpe@intel.com> writes:

Christina> +Method(
Christina> + comment="""
Christina> +If possible, return the shadow stack pointer. On some architectures, the
Christina> +shadow stack pointer is available even if the feature is disabled. To
Christina> +return the shadow stack enablement state configure SHADOW_STACK_ENABLED.
Christina> +""",

This wasn't super clear to me, but reading patch 11 made me believe that
the API is to set shadow_stack_enabled to true if shadow stacks are
enabled. Is that correct? If so could you update the comment to say
that?

You don't have to re-send the whole series just for this change fwiw.

thanks,
Tom