On Tue, Mar 07, 2017 at 09:55:19AM +0100, Richard Biener wrote:
> > --- gcc/dwarf2out.h.jj 2017-01-01 12:45:37.000000000 +0100
> > +++ gcc/dwarf2out.h 2017-03-06 12:13:48.809589411 +0100
> > @@ -163,7 +163,7 @@ enum dw_val_class
> > /* Describe a floating point constant value, or a vector constant value.
> > */
> >
> > struct GTY(()) dw_vec_const {
> > - unsigned char * GTY((atomic)) array;
> > + void * GTY((atomic)) array;
>
> Please add a comment here why this is void *. I wonder if we can't
I've already committed the patch, but will add some comment.
> tell gengtype
> how to compute the length though via some GTY attribute?
Maybe. The documentation mentions that length attribute doesn't
apply to GC allocations that don't contain pointers, maybe we could change
that. But my gengtype knowledge is very limited.
Jakub
