Re: [PATCH v3 00/13] C99 variable length array support

Joel Brobecker
2014-01-15T11:41:40Z

Hello,

> this patch series (v3) add C99 variable length support to gdb.
>
> It allows the user to evaluate a vla like an ordinary static array e.g. print
> its elements instead of printing the pointer to the array. In addition
> the size of a vla can be retrieved with gdbs builtin sizeof operator.

We've started working internally at AdaCore on getting rid of some
of the "GNAT encodings", with the goal of replacing them with pure
dwarf constructs. One of the issues we've noticed is precisely
the lack of support for dynamically-sized arrays, and I think
this patch series would be a very good stepping stone towards that.

I remember I had one patch that made me a little nervous (patch #5/13,
adding type re-fecthing after some value_* calls), but I also said
that I would be OK if the patch went in as is. Apart from that,
it seems to me that most/all? comments have been addressed?
Perhaps, all we're missing is just v4 of the patch series?

Any chance we could make progress on those? If there is anything
I can do to assist, please let me know as well!

Thank you :)
--
Joel