[PATCH] [PR tdep/20928] Decode properly flags of %ccr register on sparc64

Ivo Raisr ivo.raisr@oracle.com
Mon Mar 20 15:55:00 GMT 2017



On 17.3.2017 17:42, Pedro Alves wrote:
> On 03/14/2017 08:37 AM, Ivo Raisr wrote:
>> Ping II.
>> Can I get +1 for this small patch?
>> I.
>
> I was hoping that someone who actually knows sparc would
> take a look.  :-)
>
> My only comment would be wondering about a adding some test.
> E.g., some small assembly test in gdb.arch/ that you'd single-step
> and make sure that the expected changes can be observed in
> the affected registers / flags.

Please see attached new patch extended with new tests added.

Only the following files were added (compared to previous patch submission):
     gdb/testsuite/gdb.arch/sparc64-regs.S
     gdb/testsuite/gdb.arch/sparc64-regs.exp

I.
-------------- next part --------------
2017-03-20  Ivo Raisr  <ivo.raisr@oracle.com>

	PR tdep/20928
	* gdb/sparc-tdep.h (gdbarch_tdep) <sparc64_ccr_type>: New field.
	* gdb/sparc64-tdep.c (sparc64_ccr_type): New function.
	(sparc64_fsr_type): Fix %fsr decoding.
-------------- next part --------------
2017-03-20  Ivo Raisr  <ivo.raisr@oracle.com>

	PR tdep/20928
	* gdb.arch/sparc64-regs.exp: New file.
	* gdb.arch/sparc64-regs.S: Likewise.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: ccr_decoding.patch
Type: text/x-patch
Size: 11332 bytes
Desc: not available
URL: <http://sourceware.org/pipermail/gdb-patches/attachments/20170320/6aa038b1/attachment.bin>


More information about the Gdb-patches mailing list