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 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.
> ---
> gcc/doc/invoke.texi | 4 ++++
> gcc/final.c | 4 ++--
> gcc/opts.c | 20 ++++++++++++++++++++
> gcc/testsuite/gcc.target/i386/pr84310-2.c | 10 ++++++++++
> gcc/testsuite/gcc.target/i386/pr84310.c | 8 ++++++++
> 5 files changed, 44 insertions(+), 2 deletions(-)
> create mode 100644 gcc/testsuite/gcc.target/i386/pr84310-2.c
> create mode 100644 gcc/testsuite/gcc.target/i386/pr84310.c
>
>