On Tue, Jan 21, 2020 at 05:20:51PM +0000, Joel Hutton wrote:
> 2020-01-21 Joel Hutton <joel.hut...@arm.com\>
>
> * ira.c (ira): Revert use of simplified LRA algorithm.
>
> gcc/testsuite/ChangeLog:
>
> 2020-01-21 Joel Hutton <joel.hut...@arm.com\>
>
> PR bug/93221
> * gcc.target/aarch64/pr93221.c: New test.
Not a review, just nitpicking. Please avoid the backslash before >.
The PR number should go to both gcc/ and gcc/testsuite/ ChangeLog
entry and we don't have bug/ category; the bug is target category,
so it should be PR target/93221, or could be reclassified first in
bugzilla e.g. to middle-end and then written as PR middle-end/93221.
> @@ -0,0 +1,10 @@
> +/* PR bug/93221 */
Likewise here.
> +/* { dg-do compile } */
> +/* { dg-options "-O0 -mno-omit-leaf-frame-pointer" } */
> +
> +struct S { __Int32x4_t b[2]; };
> +
> +void __attribute__((optimize (0)))
Not sure if I understand why do you need optimize (0) attribute when the
whole test is compiled with -O0. Doesn't it ICE without the attribute
too?
> +foo (struct S x)
> +{
> +}
Jakub
