This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [RFC] Add support for Renesas RX architecture
- From: Joel Brobecker <brobecker at adacore dot com>
- To: Kevin Buettner <kevinb at redhat dot com>
- Cc: gdb-patches at sourceware dot org
- Date: Fri, 4 Dec 2009 09:38:37 -0800
- Subject: Re: [RFC] Add support for Renesas RX architecture
- References: <20091203161906.7f7a6c06@redhat.com>
> I'll note up front that there's one #if 0 in rx-tdep.c. I'd prefer to
> leave it there until Nick and I get all of the kinks worked out of
> the debug info. I'm willing to remove it though if there are objections
> to having it in new code.
I usually prefer to avoid #if 0 code, but the comment is definitely
useful. In this particular case, I don't think it's so bad, although,
IMO, the same effect could have been achieved without the #if 0, just
the comment (enabling the dwarf2 unwinders is standard, so it's not
like we need the reminder). Anyway, I'm personally OK, since I presume
that this in the TODO list to get fixed as some point in the near/medium
future.
> Comments?
>
> * configure.tgt: Add rx-*-elf target.
> * rx-tdep.c: New target.
Yes. Great code, very clean. Please go ahead and commit. I think this
also deserves a NEWS entry (you can send it as a separate patch for Eli).
Nice to see someone using the prologue-value infrastructure too, btw.
--
Joel