This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [patch] Convert frame_stash to a hash table
- From: Pedro Alves <palves at redhat dot com>
- Cc: Phil Muldoon <pmuldoon at redhat dot com>, "gdb-patches at sourceware dot org" <gdb-patches at sourceware dot org>
- Date: Thu, 16 May 2013 19:44:28 +0100
- Subject: Re: [patch] Convert frame_stash to a hash table
- References: <5194DA88 dot 6020705 at redhat dot com> <5194E257 dot 4010807 at redhat dot com> <5194E424 dot 9090605 at redhat dot com> <5194EBEF dot 9040209 at redhat dot com>
On 05/16/2013 03:23 PM, Pedro Alves wrote:
> I'd expected that a simple filter (like I imagine yours was)
> you'd not see any performance hit.
For the archives -- Phil and I spent a bit investigating tihs.
This expectation was just too naive.
The way the current frame stash is managed today (the last
frame get_frame_id is called on) lends itself to all sorts
of random places in the frame code where the stash happens
to flip to the wrong frame, or cases where we'd expect
the stash to be set but isn't (e.g., frapy_older
leaves without the stash set to the older frame). This
is just too fragile for this use case. Let's move on with the
new hash stash and be done with it.
Thanks,
--
Pedro Alves