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 alignment. >> 2) thus I also documented and limited the maximum value of -falign-* options. >> >> Patch can bootstrap on ppc64le-redhat-linux and survives regression tests. >> i386.exp test-suite works fine on x86_64 machine. >> >> Ready to be installed? >> Martin >> >> gcc/ChangeLog: >> >> 2018-02-09 Martin Liska <mli...@suse.cz> >> >> PR c/84310 >> PR target/79747 >> * final.c (shorten_branches): Build align_tab array with one >> more element. >> * opts.c (finish_options): Add alignment option limit check. >> (MAX_CODE_ALIGN): Likewise. >> (MAX_CODE_ALIGN_VALUE): Likewise. >> * doc/invoke.texi: Document maximum allowed option value for >> all -falign-* options. >> >> gcc/testsuite/ChangeLog: >> >> 2018-02-12 Martin Liska <mli...@suse.cz> >> >> PR c/84310 >> PR target/79747 >> * gcc.target/i386/pr84310.c: New test. >> * gcc.target/i386/pr84310-2.c: Likewise. > > Ok, thanks. > > Jakub >
Thanks Jakub! Would it be possible to backport that to active branches? Martin