This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
RE: [patch v8 15/24] frame, backtrace: allow targets to supply a frame unwinder
- From: "Metzger, Markus T" <markus dot t dot metzger at intel dot com>
- To: Pedro Alves <palves at redhat dot com>
- Cc: "jan dot kratochvil at redhat dot com" <jan dot kratochvil at redhat dot com>, "gdb-patches at sourceware dot org" <gdb-patches at sourceware dot org>
- Date: Tue, 17 Dec 2013 08:26:00 +0000
- Subject: RE: [patch v8 15/24] frame, backtrace: allow targets to supply a frame unwinder
- Authentication-results: sourceware.org; auth=none
- References: <1386839747-8860-1-git-send-email-markus dot t dot metzger at intel dot com> <1386839747-8860-16-git-send-email-markus dot t dot metzger at intel dot com> <52AB5194 dot 2050205 at redhat dot com> <A78C989F6D9628469189715575E55B230AA38BB1 at IRSMSX104 dot ger dot corp dot intel dot com> <52AF4E2C dot 2060408 at redhat dot com>
> -----Original Message-----
> From: Pedro Alves [mailto:palves@redhat.com]
> Sent: Monday, December 16, 2013 8:02 PM
> >> ... parts look a bit unrelated. It'd be good to have
> >> the rationale for this spelled out. I can understand
> >> why they might be necessary, but I don't see why
> >> target vs non-unwinders unwinders would be special here?
> >
> > Those changes are not directly related to target vs non-target
> > unwinders. They are connected in that they allow new types
> > of unwinders.
>
> >
> > I spelled out the rationale for each change in the commit
> > message.
>
> If I missed it, please point me at it. E.g., I didn't see a
> rationale for the get_frame_unwind_stop_reason change.
> Something like "We need to do X in get_frame_unwind_stop_reason
> because otherwise, when we do Y, W doesn't work."
I meant that based on your review, I now spell out the rationale
for each change in the commit message. I have not sent the updated
patch, yet.
> > Should I further split this patch into three?
>
> IMO, yes.
OK. I'll do that, as well.
Regards,
Markus.
Intel GmbH
Dornacher Strasse 1
85622 Feldkirchen/Muenchen, Deutschland
Sitz der Gesellschaft: Feldkirchen bei Muenchen
Geschaeftsfuehrer: Christian Lamprechter, Hannes Schwaderer, Douglas Lusk
Registergericht: Muenchen HRB 47456
Ust.-IdNr./VAT Registration No.: DE129385895
Citibank Frankfurt a.M. (BLZ 502 109 00) 600119052