Re: [PATCH] handle multiple flexible array members (PR 83588)

2018-01-15 Thread Jason Merrill
On Mon, Jan 15, 2018 at 10:05 PM, Martin Sebor wrote: > On 01/15/2018 07:10 PM, Jason Merrill wrote: >> >> OK. > > Thanks. I keep forgetting to get approval to backport these > simple bug fixes. Is this one okay for the 7 and 6 branches? Yes. Jason

Re: [PATCH] handle multiple flexible array members (PR 83588)

2018-01-15 Thread Martin Sebor
On 01/15/2018 07:10 PM, Jason Merrill wrote: OK. Thanks. I keep forgetting to get approval to backport these simple bug fixes. Is this one okay for the 7 and 6 branches? Martin On Sun, Jan 14, 2018 at 6:47 PM, Martin Sebor wrote: The attached patch fixes PR c++/83588 - struct with two f

Re: [PATCH] handle multiple flexible array members (PR 83588)

2018-01-15 Thread Jason Merrill
OK. On Sun, Jan 14, 2018 at 6:47 PM, Martin Sebor wrote: > The attached patch fixes PR c++/83588 - struct with two flexible > arrays causes an internal compiler error. The ICE is caused by > the same assertion in varasm.c that has led to other similar > reports in the past: > > /* Given a non-

[PATCH] handle multiple flexible array members (PR 83588)

2018-01-14 Thread Martin Sebor
The attached patch fixes PR c++/83588 - struct with two flexible arrays causes an internal compiler error. The ICE is caused by the same assertion in varasm.c that has led to other similar reports in the past: /* Given a non-empty initialization, this field had better be last. Given a fl