{Spam?} Re: [PATCH][PR symtab/32658] Fix parsing .debug_aranges section for MIPS signed addresses

[Martin Simmons <qqxnjvamvxwx@dyxyl.com>]

>>>>> On Sat, 08 Mar 2025 09:39:06 -0700, Tom Tromey said:
> 
>>>>> "Martin" == Martin Simmons <qqxnjvamvxwx@dyxyl.com> writes:
> 
> Martin> This patch fixes https://sourceware.org/bugzilla/show_bug.cgi?id=32658 by sign
> Martin> extending the addresses read from the .debug_aranges section as required for
> Martin> MIPS.  I made it explicitly sign extend the addresses after they are read
> Martin> (instead of using extract_signed_integer) because I don't know if the
> Martin> address_size in the section will always be equal to the gdb arch_size.
> 
> I wonder if perhaps the relevant gdbarch method should be used instead.

Do you mean gdbarch_integer_to_address like in
dwarf_expr_context::fetch_address?  I considered that, but it looked
like a kludge and it wasn't clear to me if that would be good for avr
(calling avr_integer_to_address).

Also, the old parser used comp_unit_head::read_address (called from
dwarf_decode_lines_1) so that didn't do anything with gdbarch.

__Martin