On Wed, Jul 08, 2015 at 02:40:33PM +0200, Richard Biener wrote:
> > toplev.c already sets maximum_field_alignment directly, and to a different
> > value:
> > maximum_field_alignment = initial_max_fld_align * BITS_PER_UNIT;
> > so I'm not sure you need a new function. And, shouldn't you reset to
> > maximum_field_alignment = initial_max_fld_align * BITS_PER_UNIT;
> > ? -fpack-struct= is an ABI changing option, so IMHO it should affect
> > even gcov_info_type etc.
>
> That would support the 2nd patch variant. Note that -fpack-struct
> can be specified in optimize attributes...
Yeah, the second patch LGTM.
Jakub
