This is the mail archive of the
mailing list for the GDB project.
Re: [FYI] Inlining support, rough patch
- From: Daniel Jacobowitz <drow at false dot org>
- To: Jan Kratochvil <jan dot kratochvil at redhat dot com>
- Cc: Mark Kettenis <mark dot kettenis at xs4all dot nl>, tromey at redhat dot com, gdb-patches at sourceware dot org
- Date: Sat, 27 Jun 2009 14:01:22 -0400
- Subject: Re: [FYI] Inlining support, rough patch
- References: <20080613152754.GA4220@caradoc.them.org> <20080715192020.GB3094@caradoc.them.org> <200807172353.m6HNr1nY015884@brahms.sibelius.xs4all.nl> <20080718130308.GA19045@caradoc.them.org> <200807251446.m6PEkfwc027635@brahms.sibelius.xs4all.nl> <20080725174636.GB2433@caradoc.them.org> <firstname.lastname@example.org> <200903312042.n2VKgIUx003764@brahms.sibelius.xs4all.nl> <20090420154909.GA5386@caradoc.them.org> <20090420155405.GA6072@host0.dyn.jankratochvil.net>
On Mon, Apr 20, 2009 at 05:54:05PM +0200, Jan Kratochvil wrote:
> On Mon, 20 Apr 2009 17:49:09 +0200, Daniel Jacobowitz wrote:
> > Nothing algorithmic has changed,
> > but we have a few bug fixes and improved testcases.
> Fedora also has various fixes on top of it:
Thanks for the link. Some of these I've already got fixes for.
For instance, the changes in breakpoint.c about returning from a
function to an inlined frame; I fixed it somewhere else, but we
did discover that problem.
Others, particularly the testsuite changes, I don't understand. I'd
need to see a compiler that failed to work out why the changes were
needed. So if you still need patches after the latest version of the
patch is checked in, please let me know and I'll reproduce the
current_pc_is_notcurrent is interesting. Do I have the scenario
* function() calls inlined() calls other()
* finish from other()
* show the end of inlined() instead of the next line of function()
I can't figure out if we should do this or not. It does seem useful.
But that's not where we are; we're showing the previous call site
instead of the next instruction.
I think we should consider it as a general change for finish instead
of specific to inlining. The comments in your patch suggested that
I merged the read_type_die fix, thanks.