[RFA 3/5] New port: CR16: gdb port
Joel Brobecker
brobecker@adacore.com
Thu Nov 22 17:50:00 GMT 2012
> > + allWords =
> > Can you please void using the mixed-cap style for variables?
> This variable along with currInsn suffer from the same problem. However, they
> have been borrowed from existing code from opcodes/cr16-dis.c
> May I leave them here as is for now as it is used at several places in cr16-dis.c?
Yes. I thought it was a variable declared inside GDB. This is fine.
> 2012-11-20 Kaushik Phatak <kaushik.phatak@kpitcummins.com>
>
> gdb/Changelog
> * configure.tgt: Handle cr16*-*-*linux and cr16*-*-*.
> * cr16-linux-tdep.c: New file.
> * cr16-tdep.c: New file.
> * cr16-tdep.h: New file.
A couple of very very minor nits. Pre-approved with the changes
requested below. Once you get approval for the BFD bits, you may
commit this patch. When you do, can you send us a notification,
as well as a copy of the patch that you commit (because it is
going to be different from the one posted here).
> + if (p->has_frame_ptr)
> + {
> + CORE_ADDR fp = get_frame_register_unsigned (this_frame, CR16_FP_REGNUM);
> + return fp - p->frame_ptr_offset;
> + }
> + else
> + {
> + CORE_ADDR sp = get_frame_register_unsigned (this_frame, CR16_SP_REGNUM);
> + return sp - p->frame_size;
> + }
Missing empty line after variable declaration in both blocks.
> +};
Can you remov the trailing spaces?
Thank you,
--
Joel
More information about the Gdb-patches
mailing list