]> git.ipfire.org Git - thirdparty/gcc.git/commitdiff
[testsuite] Fix guality/pr45882.c for flto
authorTom de Vries <tdevries@suse.de>
Thu, 21 Jun 2018 13:37:59 +0000 (13:37 +0000)
committerTom de Vries <vries@gcc.gnu.org>
Thu, 21 Jun 2018 13:37:59 +0000 (13:37 +0000)
Atm this test in pr45882.c fails:
...
  int d = a[i];  /* { dg-final { gdb-test 16 "d" "112" } } */
...
as follows:
...
FAIL: gcc.dg/guality/pr45882.c   -O2 -flto -fuse-linker-plugin \
      -fno-fat-lto-objects  line 16 d == 112
...

In more detail, gdb fails to print the value of d:
...
Breakpoint 1, foo (i=i@entry=7, j=j@entry=7) at pr45882.c:16
16        ++v;
$1 = <optimized out>
$2 = 112
<optimized out> != 112
...

Variable d is a local variable in function foo, initialized from global array a.
When compiling, first cddce1 removes the initialization of d in foo, given
that d is not used afterwards.  Then ipa marks array a as write-only, and
removes the stores to array a in main.  This invalidates the location
expression for d, which points to a[i], so it is removed, which is why gdb
ends up printing <optimized out> for d.

This patches fixes the fail by adding attribute used to array a, preventing
array a from being marked as write-only.

Tested on x86_64.

2018-06-21  Tom de Vries  <tdevries@suse.de>

* gcc.dg/guality/pr45882.c (a): Add used attribute.

From-SVN: r261845

gcc/testsuite/ChangeLog
gcc/testsuite/gcc.dg/guality/pr45882.c

index 93ba5e10f5cb6884ae4ecd8a2c80a2fed748c692..977a22d0bf6995b1b15f5d03b409d7b43d5715f1 100644 (file)
@@ -1,3 +1,7 @@
+2018-06-21  Tom de Vries  <tdevries@suse.de>
+
+       * gcc.dg/guality/pr45882.c (a): Add used attribute.
+
 2018-06-21  Tom de Vries  <tdevries@suse.de>
 
        PR tree-optimization/85859
index 5ca22d4f4ad4c18e25596838b384982cb0e204e0..ece35238a301e94a3b3deda3729064c61a19beef 100644 (file)
@@ -3,7 +3,7 @@
 /* { dg-options "-g" } */
 
 extern void abort (void);
-int a[1024];
+int a[1024] __attribute__((used));
 volatile short int v;
 
 __attribute__((noinline,noclone,used)) int