Re: [PATCH][gdb/testsuite] Support .debug_aranges in dwarf assembly

[Simon Marchi via Gdb-patches <gdb-patches@sourceware.org>]

On 2021-08-27 12:14 p.m., Tom Tromey wrote:
>>>>>> "Simon" == Simon Marchi <simon.marchi@polymtl.ca> writes:
> 
> Simon> Could we consistently use the
> Simon> "options" pattern, such as the one used by aranges and cu?
> 
> Yeah, that would be fine by me.
> 
>>> This is nested in 'aranges', but that's weird style in Tcl.
>>> It redefines the proc every time 'aranges' is invoked.
>>> I think it's better to just namespace scope this.
> 
> Simon> But doing it this way makes it so that you can only invoke arange when
> Simon> you are in aranges' body, isn't that useful?  I guess the downside to
> Simon> redefining the proc everytime is performance, but that's really not a
> Simon> concern here (it runs quickly enough).
> 
> To do that, it also have to delete the 'arange' proc after evaluating
> the body.  I suppose that would be alright by me.

Really?  This technique is used in proc rnglists, and that doesn't seem
to cause a problem.

Simon