Re: [PATCH] Add limit for maximal alignment options (PR c/84310).

2018-02-20 Thread Jakub Jelinek
On Tue, Feb 20, 2018 at 11:05:50AM +0100, Martin Liška wrote: > Thanks Jakub! > Would it be possible to backport that to active branches? Yes, but give it some time on the trunk first. Jakub

Re: [PATCH] Add limit for maximal alignment options (PR c/84310).

2018-02-20 Thread Martin Liška
On 02/20/2018 09:43 AM, Jakub Jelinek wrote: > On Mon, Feb 12, 2018 at 01:09:43PM +0100, Martin Liška wrote: >> Following patch fixes 2 issues with -falign-*: >> 1) when using -malign-x=16 (or corresponding -falign-* value) then ICE >> appeared >> as code in final.c can deal just with limited

Re: [PATCH] Add limit for maximal alignment options (PR c/84310).

2018-02-20 Thread Jakub Jelinek
On Mon, Feb 12, 2018 at 01:09:43PM +0100, Martin Liška wrote: > Following patch fixes 2 issues with -falign-*: > 1) when using -malign-x=16 (or corresponding -falign-* value) then ICE > appeared > as code in final.c can deal just with limited alignment. > 2) thus I also documented and limited the

Re: [PATCH] Add limit for maximal alignment options (PR c/84310).

2018-02-20 Thread Martin Liška
PING^1 On 02/12/2018 01:09 PM, Martin Liška wrote: > Hi. > > Following patch fixes 2 issues with -falign-*: > 1) when using -malign-x=16 (or corresponding -falign-* value) then ICE > appeared > as code in final.c can deal just with limited alignment. > 2) thus I also documented and limited the

[PATCH] Add limit for maximal alignment options (PR c/84310).

2018-02-12 Thread Martin Liška
Hi. Following patch fixes 2 issues with -falign-*: 1) when using -malign-x=16 (or corresponding -falign-* value) then ICE appeared as code in final.c can deal just with limited alignment. 2) thus I also documented and limited the maximum value of -falign-* options. Patch can bootstrap on