[PATCH] PR92090: Fix testcase failures by r276469

[Segher Boessenkool]

Hi!

On Wed, Oct 30, 2019 at 02:02:44AM -0500, Xiong Hu Luo wrote:
> -finline-functions is enabled by default for O2 since r276469, update the
> test cases that inline small functions caused instruction number difference.

> --- a/gcc/testsuite/gcc.target/powerpc/pr79439-1.c
> +++ b/gcc/testsuite/gcc.target/powerpc/pr79439-1.c
> @@ -22,7 +22,7 @@ rec (int a)
>    return a + ret;
>  }
>  
> -/* { dg-final { scan-assembler-times {\mbl f\M}   1 } } */
> -/* { dg-final { scan-assembler-times {\mbl g\M}   1 } } */
> +/* { dg-final { scan-assembler-times {\mbl f\M}   2 } } */
> +/* { dg-final { scan-assembler-times {\mbl g\M}   2 } } */
>  /* { dg-final { scan-assembler-times {\mbl rec\M} 1 } } */
> -/* { dg-final { scan-assembler-times {\mnop\M}    2 } } */
> +/* { dg-final { scan-assembler-times {\mnop\M}    4 } } */

It's probably better to disable inlining for this one.

> diff --git a/gcc/testsuite/gcc.target/powerpc/vsx-builtin-7.c b/gcc/testsuite/gcc.target/powerpc/vsx-builtin-7.c
> index 5d31309f272..66b526832eb 100644
> --- a/gcc/testsuite/gcc.target/powerpc/vsx-builtin-7.c
> +++ b/gcc/testsuite/gcc.target/powerpc/vsx-builtin-7.c
> @@ -192,7 +192,7 @@ vector unsigned __int128 splat_uint128 (unsigned __int128 x) { return vec_splats
>  
>  /* { dg-final { scan-assembler-times {\mrldic\M} 0  { target { be && ilp32 } } } } */
>  /* { dg-final { scan-assembler-times {\mrldic\M} 64 { target { be && lp64 } } } } */
> -/* { dg-final { scan-assembler-times {\mrldic\M} 64 { target le } } } */
> +/* { dg-final { scan-assembler-times {\mrldic\M} 65 { target le } } } */
>  /* { dg-final { scan-assembler-times "xxpermdi" 4 { target be } } } */
>  /* { dg-final { scan-assembler-times "xxpermdi" 6 { target le } } } */
>  /* { dg-final { scan-assembler-times "vspltisb" 2 } } */

This one, too.

(Also needs testing on BE btw -- it's often fine if you don't, but only
fixing the tests for LE isn't great ;-) )


Segher