Re: [PATCH] gdb: use std::vector instead of alloca in core_target::get_core_register_section

Simon Marchi
2020-01-13T19:34:31Z

On 2020-01-13 12:30 p.m., Tom Tromey wrote:
>>>>>> "Simon" == Simon Marchi <simon.marchi@polymtl.ca> writes:
>
> Simon> We are using alloca to hold the contents of a the core register
> Simon> sections. These sections are typically fairly small, but there is no
> Simon> realy guarantee, so I think it would be more reasonable to just use
> Simon> dynamic allocation here.
>
> I would be fine with simply avoiding alloca in most, or all, cases.

Agreed.

> Simon> gdb/ChangeLog:
>
> Simon> * corelow.c (core_target::get_core_register_section): Use
> Simon> std::vector instead of alloca.
>
> Looks reasonable to me.

Ok, thanks all, I've pushed this patch.

Simon