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
