Re: [PATCH] Fix handling of variable length fields in structures (PR c/82210)

2018-02-13 Thread Jeff Law
On 02/08/2018 11:22 PM, Jakub Jelinek wrote: > Hi! > > When placing a variable length field into a structure, we need to update > rli->offset_align for the next field. We do: > rli->offset_align = MIN (rli->offset_align, desired_align); > which updates it according to the start of that VLA field,

[PATCH] Fix handling of variable length fields in structures (PR c/82210)

2018-02-08 Thread Jakub Jelinek
Hi! When placing a variable length field into a structure, we need to update rli->offset_align for the next field. We do: rli->offset_align = MIN (rli->offset_align, desired_align); which updates it according to the start of that VLA field, the problem is that if the field doesn't have a size tha