This is the mail archive of the
gdb@sourceware.org
mailing list for the GDB project.
RE: a review and questions on avr_scan_prologue()
- From: "Weddington, Eric" <Eric dot Weddington at atmel dot com>
- To: Petr Hluzín <petr dot hluzin at gmail dot com>, <gdb at sourceware dot org>
- Cc: "Tristan Gingold" <gingold at adacore dot com>, "Joerg Wunsch" <joerg_wunsch at uriah dot heep dot sax dot de>
- Date: Mon, 15 Feb 2010 22:12:43 -0700
- Subject: RE: a review and questions on avr_scan_prologue()
- References: <6a6f635a1002131556sc428adfu6033489930eca7b6@mail.gmail.com>
> -----Original Message-----
> From: Petr Hluzín [mailto:petr.hluzin@gmail.com]
> Sent: Sunday, February 14, 2010 5:27 AM
> To: gdb@sourceware.org
> Cc: Tristan Gingold
> Subject: a review and questions on avr_scan_prologue()
>
> Hello
>
> I took a look at avr-tdep.c [1] and I found some places which are
> either bug or are not clear to me.
Could you please at least fill out bug reports for these 2 issues? Please put me on the CC list. That way they're recorded.
> Also:
> The avr_scan_prologue()'s recognizes several well-known prologues. Is
> there a reason why it does not use the general prologue analysis
> algorithm as described in the documentation [2]?
IIRC, the AVR GCC prologue and epilogue used to be some fixed code. Recently the AVR GCC port generates RTL-based prologues and epiloges. I'm sure AVR GDB has not kept up.
> I think universal prologue analysis is quite easy with AVR arch. The
> code might be shorter (though less clear).
> I might try to write the code if you are interested.
> (The current prologue scan code chokes on hand-crafted assembly.)
Yes, we would be very interested.
Eric Weddington