Hi Ulrich and community,

Please find attached the patch. {See: 0001-Enable-vector-instruction-debugging-for-AIX.patch}.

Please see my reply to your comments marked in blue.

Kindly push this patch if there are no further changes.

Have a nice day ahead.

Thanks and regards,
Aditya.

>+    /* VSX registers.  */
>+    if (tdep->ppc_vsr0_upper_regnum != -1 && (regno == -1
>+       || (regno >=tdep->ppc_vsr0_upper_regnum
>+       && regno < tdep->ppc_vsr0_upper_regnum + ppc_num_vshrs)))

Here it is in this patch.

From: Ulrich Weigand <Ulrich.Weigand@de.ibm.com>
Date: Tuesday, 7 March 2023 at 6:25 PM
To: gdb-patches@sourceware.org <gdb-patches@sourceware.org>, Aditya Kamath1 <Aditya.Kamath1@ibm.com>, simark@simark.ca <simark@simark.ca>
Cc: Sangamesh Mallayya <sangamesh.swamy@in.ibm.com>
Subject: Re: [PATCH] Enable vector instruction debugging for AIX
Aditya Kamath1 <Aditya.Kamath1@ibm.com> wrote:

>Kindly push this patch if there are no changes.

This one issue still isn't addressed:

>>+    /* VSX registers.  */
>>+    if (tdep->ppc_vsr0_upper_regnum != -1 && (regno == -1
>>+      || (regno >=tdep->ppc_vsr0_upper_regnum)))
>
>You've now removed the upper limit completely.  I think this needs an
>  && regno < tdep->ppc_vsr0_upper_regnum + ppc_num_vshrs

Otherwise, everything looks good to me now.

Bye,
Ulrich