From c5efca34f357db43a814748a5ee4f58fd10d193e Mon Sep 17 00:00:00 2001 From: Jakub Jelinek Date: Wed, 16 Jun 2021 13:10:48 +0200 Subject: [PATCH] testsuite: Use noipa attribute instead of noinline, noclone I've noticed this test now on various arches sometimes FAILs, sometimes PASSes (the line 12 test in particular). The problem is that a = 0; initialization in the caller no longer happens before the f(&a) call as what the argument points to is only used in debug info. Making the function noipa forces the caller to initialize it and still tests what the test wants to test, namely that we don't consider *p as valid location for the c variable at line 18 (after it has been overwritten with *p = 1;). 2021-06-16 Jakub Jelinek * gcc.dg/guality/pr49888.c (f): Use noipa attribute instead of noinline, noclone. (cherry picked from commit a490b1dc0b3c26bff2ee00ac0da2d606d2009e3a) --- gcc/testsuite/gcc.dg/guality/pr49888.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/gcc/testsuite/gcc.dg/guality/pr49888.c b/gcc/testsuite/gcc.dg/guality/pr49888.c index 4f3a2501eb7c..919cfc9c7693 100644 --- a/gcc/testsuite/gcc.dg/guality/pr49888.c +++ b/gcc/testsuite/gcc.dg/guality/pr49888.c @@ -4,7 +4,7 @@ static int v __attribute__((used)); -static void __attribute__((noinline, noclone)) +static void __attribute__((noipa)) f (int *p) { int c = *p; -- 2.47.2