diff mbox series

testsuite: Increase auto-inlining param in gcc.dg/ipa/remref-7.c (PR 105183)

Message ID ri6zgkvalq8.fsf@suse.cz
State New
Headers show
Series testsuite: Increase auto-inlining param in gcc.dg/ipa/remref-7.c (PR 105183) | expand

Commit Message

Martin Jambor April 8, 2022, 10:54 a.m. UTC
Hi,

a scan dump of testsuite gcc.dg/ipa/remref-7.c fails on a number of
platforms.  I investigated only i?86-*-* with -mno-sse but assume the
issue is the same on all of the affected platform.

Because function bar is not inlined there even though it is only called
once, the process that is being tested is simply not triggered.  This
can be "fixed" by increasing parameter max-inline-insns-auto to
something high, I randomly picked 100.

I have only manually tested the change but hopefully that is enough.
Of course, anyone who can quickly test the patch on another affected
platform is welcomed to do so.

OK for master?

Thanks,

Martin


gcc/testsuite/ChangeLog:

2022-04-08  Martin Jambor  <mjambor@suse.cz>

	PR testsuite/105183
	* gcc.dg/ipa/remref-7.c: Add --param max-inline-insns-auto=100 to options.
---
 gcc/testsuite/gcc.dg/ipa/remref-7.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Richard Biener April 8, 2022, 11:40 a.m. UTC | #1
On Fri, Apr 8, 2022 at 12:55 PM Martin Jambor <mjambor@suse.cz> wrote:
>
> Hi,
>
> a scan dump of testsuite gcc.dg/ipa/remref-7.c fails on a number of
> platforms.  I investigated only i?86-*-* with -mno-sse but assume the
> issue is the same on all of the affected platform.
>
> Because function bar is not inlined there even though it is only called
> once, the process that is being tested is simply not triggered.  This
> can be "fixed" by increasing parameter max-inline-insns-auto to
> something high, I randomly picked 100.
>
> I have only manually tested the change but hopefully that is enough.
> Of course, anyone who can quickly test the patch on another affected
> platform is welcomed to do so.
>
> OK for master?

OK

> Thanks,
>
> Martin
>
>
> gcc/testsuite/ChangeLog:
>
> 2022-04-08  Martin Jambor  <mjambor@suse.cz>
>
>         PR testsuite/105183
>         * gcc.dg/ipa/remref-7.c: Add --param max-inline-insns-auto=100 to options.
> ---
>  gcc/testsuite/gcc.dg/ipa/remref-7.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/gcc/testsuite/gcc.dg/ipa/remref-7.c b/gcc/testsuite/gcc.dg/ipa/remref-7.c
> index b2c26ab7fd5..152b9d0ca94 100644
> --- a/gcc/testsuite/gcc.dg/ipa/remref-7.c
> +++ b/gcc/testsuite/gcc.dg/ipa/remref-7.c
> @@ -1,5 +1,5 @@
>  /* { dg-do compile } */
> -/* { dg-options "-O2 -fno-early-inlining -fno-ipa-sra -fdump-ipa-inline"  }  */
> +/* { dg-options "-O2 -fno-early-inlining -fno-ipa-sra -fdump-ipa-inline --param max-inline-insns-auto=100" }  */
>
>  int rglobal = 0;
>  int g;
> --
> 2.35.1
>
diff mbox series

Patch

diff --git a/gcc/testsuite/gcc.dg/ipa/remref-7.c b/gcc/testsuite/gcc.dg/ipa/remref-7.c
index b2c26ab7fd5..152b9d0ca94 100644
--- a/gcc/testsuite/gcc.dg/ipa/remref-7.c
+++ b/gcc/testsuite/gcc.dg/ipa/remref-7.c
@@ -1,5 +1,5 @@ 
 /* { dg-do compile } */
-/* { dg-options "-O2 -fno-early-inlining -fno-ipa-sra -fdump-ipa-inline"  }  */
+/* { dg-options "-O2 -fno-early-inlining -fno-ipa-sra -fdump-ipa-inline --param max-inline-insns-auto=100" }  */
 
 int rglobal = 0;
 int g;