This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [RFA 08/15] Remove readin and compunit_symtab fields from psymtab
- From: Tom Tromey <tom at tromey dot com>
- To: Simon Marchi <simark at simark dot ca>
- Cc: Tom Tromey <tom at tromey dot com>, gdb-patches at sourceware dot org
- Date: Sun, 07 Oct 2018 18:01:15 -0600
- Subject: Re: [RFA 08/15] Remove readin and compunit_symtab fields from psymtab
- References: <20180510222357.27332-1-tom@tromey.com> <20180510222357.27332-9-tom@tromey.com> <5f8f38c4-8b4b-9fea-b9f2-7c69758a09c2@simark.ca> <87lg7mi5al.fsf@tromey.com> <87zhvreujr.fsf@tromey.com> <0f59e2a3-b9e2-511c-7807-b7a68a8df1cf@simark.ca>
>>>>> "Simon" == Simon Marchi <simark@simark.ca> writes:
>> I ran the gdb.perf tests and on some tests, it seems that the results
>> are within the noise; but on backtrace (2048) and skip-function-3000 (4000)
>> gdb is about 4% slower. I don't know if this is enough to worry about.
Simon> That sounds like a reasonable trade-off in exchange for the feature of decoupling
Simon> psymtabs from objfiles. Perhaps we can put it on the list of "candidates for htab_t"
Simon> (could be a comment next to the field), for when we finally bring the C++ wrappers
Simon> from gcc :).
Just to be clear, 4% was the slowdown after I switched it to htab_t.
Tom