Re: [RFA 3/5] New port: CR16: gdb port

Joel Brobecker
2013-01-18T14:17:07Z

> Thanks for spending time on this. I have tried to address all your
> points in this updated patch.

And I confirm you did :).

> 2013-01-18 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.

Looks good to me.

One warning, though: You added src/configure.ac and src/configure
to the patch. JIC we hadn't mentioned this, this part is controlled
by GCC, and needs to be approved there. You can go ahead and commit
the GDB part.

--
Joel