[PATCH v2 0/6] [gdb/symtab] Fix data-races in gdb.base/index-cache.exp
Tom de Vries
tdevries@suse.de
Fri Aug 4 00:14:07 GMT 2023
On 8/2/23 21:44, Tom Tromey wrote:
>>>>>> "Tom" == Tom de Vries via Gdb-patches <gdb-patches@sourceware.org> writes:
>
> Thanks for this series. I read through it and sent a few notes.
>
> Tom> There's one more patch like that, I checked using pahole that the struct size
> Tom> is not increased.
>
> "ptype/o" is basically pahole FWIW.
>
> I tend to think that in most cases, the size of objects doesn't really
> matter. I mean, obviously we don't want to bloat them unnecessarily,
> but for something like this, I just wouldn't worry much... and if we did
> care there's probably some other way we could shrink them than worrying
> about packing.
>
> In the olden days pretty much the only thing really worth worrying about
> was partial symbols. Now I guess it would be cooked_index_entry.
Ack.
I've submitted a v3, following up on comments and dropping the patch
that's no longer required.
I'll commit tomorrow unless there are further comments.
Thanks,
- Tom
More information about the Gdb-patches
mailing list