This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [committed] Remove a redundant condition check in tic6x-linux-tdep.c

On 11/30/2012 08:22 PM, Mark Kettenis wrote: 
Hmm, well, perhaps the regnum variable should just be made signed.
That's typically what we use for register numbers, since -1 has the
special meaning of "all registers"/

Mark,
This patch is to change function tic6x_register_sigcontext_offset, which is to get the input REGNUM, and return the offset of corresponding register in 'struct sigcontext'. So -1 means 'all registers' doesn't apply here.

Thank for reminding me this, I don't know -1 means 'all registers' before.

--
Yao (éå)

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]