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

[Tom Tromey <tom@tromey.com>]

>>>>> "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