procyberian/gcc
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

testsuite: Add -fno-ivopts to gcc.dg/Wuse-after-free-2.c, PR108828

Browse source 
For cris-elf before this patch, ever since it was added,
this test gets:

Running /x/gcc/testsuite/gcc.dg/dg.exp ...
FAIL: gcc.dg/Wuse-after-free-2.c  (test for warnings, line 115)
FAIL: gcc.dg/Wuse-after-free-2.c  (test for warnings, line 116)

and comparing tree dumps with a native x86_64-pc-linux-gnu
run shows a suspicious difference in the "180t.ivopts" dump.
Indeed -fno-ivopts makes the warning appear for cris-elf
too.  It was suggested to simply add -fno-ivopts to the
test-flags, like before -fno-tree-loop-distribute-patterns
was added; thus.

	PR tree-optimization/108828
	* gcc.dg/Wuse-after-free-2.c: Add -fno-ivopts.
This commit is contained in:
Hans-Peter Nilsson 2023-02-24 17:22:02 +01:00 • committed by Hans-Peter Nilsson
parent bf0e0fc0ea
commit 8c58f4b798
1 changed files with 3 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
gcc/testsuite/gcc.dg/Wuse-after-free-2.c
View file

@ -1,6 +1,6 @@
/* PR middle-end/104232 - spurious -Wuse-after-free after conditional free
{ dg-do compile }
{ dg-options "-O2 -Wall -fno-tree-loop-distribute-patterns" } */
{ dg-options "-O2 -Wall -fno-tree-loop-distribute-patterns -fno-ivopts" } */
void free (void*);
@ -108,7 +108,8 @@ int warn_cond_loop (char *p)
char *q = p;
/* -fno-tree-loop-distribute-patterns ensures this does not get converted
into rawmemchr (making q and p unrelated). */
into rawmemchr (making q and p unrelated). Also, -fno-ivopts is required
for some targets, to not lose track of the pointer. */
while (*q)
++q;