On Fri, Jan 08, 2016 at 11:07:17PM +1100, Kugan wrote:
> gcc/ChangeLog:
> 
> 2016-01-08  Kugan Vivekanandarajah  <kug...@linaro.org>
>           Jim Wilson  <jim.wil...@linaro.org>
> 
>       PR target/69194
>       * config/arm/arm-builtins.c (arm_expand_neon_args): Call
>        copy_to_mode_reg instead of force_reg.

The line should be just tab indented, not tab and space.

> diff --git a/gcc/config/arm/arm-builtins.c b/gcc/config/arm/arm-builtins.c
> index 11cd17d..40dd620 100644
> --- a/gcc/config/arm/arm-builtins.c
> +++ b/gcc/config/arm/arm-builtins.c
> @@ -2146,7 +2146,7 @@ constant_arg:
>             if (!(*insn_data[icode].operand[opno].predicate)
>                     (op[argc], mode[argc]))
>               op[argc] = (replace_equiv_address
> -                         (op[argc], force_reg (Pmode, XEXP (op[argc], 0))));
> +                         (op[argc], copy_to_mode_reg (Pmode, XEXP (op[argc], 
> 0))));

and this line is now too long.

>                break;
>  
>           case NEON_ARG_STOP:
> diff --git a/gcc/testsuite/gcc.target/arm/pr69194.C 
> b/gcc/testsuite/gcc.target/arm/pr69194.C
> index e69de29..b78fb5b 100644
> --- a/gcc/testsuite/gcc.target/arm/pr69194.C
> +++ b/gcc/testsuite/gcc.target/arm/pr69194.C
> @@ -0,0 +1,504 @@
> +

Add /* PR target/69194 */
first?

> +/* { dg-do-compile } */
> +/* { dg-require-effective-target arm_neon } */
> +/* { dg-options "-O2 -mfpu=neon -S  -Wno-deprecated-declarations" } */

The -S does not belong there, nor extra spaces.
The testcase is too long, can you try to delta reduce or creduce it further?

        Jakub

Reply via email to