This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Regression (was: Re: [PATCH][4.2] Fix PR31797, make forwprop not prop...)
- From: Hans-Peter Nilsson <hans-peter dot nilsson at axis dot com>
- To: rguenther at suse dot de
- Cc: gcc-patches at gcc dot gnu dot org
- Date: Sun, 13 May 2007 01:48:08 +0200
- Subject: Regression (was: Re: [PATCH][4.2] Fix PR31797, make forwprop not prop...)
> Date: Sat, 12 May 2007 19:20:04 +0200 (CEST)
> From: Richard Guenther <rguenther@suse.de>
> 2007-05-12 Richard Guenther <rguenther@suse.de>
>
> PR tree-optimization/31797
> * tree-ssa-forwprop.c (forward_propagate_addr_expr): Do not
> propagate into a stmt that has volatile ops.
This change (*) causes the following regressions for cris-elf,
but seem general so I'm a bit surprised you don't see them on
x86_64:
Running /tmp/hpautotest-gcc1/gcc/gcc/testsuite/gcc.dg/tree-ssa/tree-ssa.exp ...
...
FAIL: gcc.dg/tree-ssa/pr17141-1.c scan-tree-dump-times & 0
...
Running /tmp/hpautotest-gcc1/gcc/gcc/testsuite/g++.dg/dg.exp ...
...
FAIL: g++.dg/tree-ssa/pr31146.C scan-tree-dump i\[j.*\] = 1;
with log files containing:
gcc.log:
Executing on host: /tmp/hpautotest-gcc1/cris-elf/gccobj/gcc/xgcc -B/tmp/hpautotest-gcc1/cris-elf/gccobj/gcc/ /tmp/hpautotest-gcc1/gcc/gcc/testsuite/gcc.dg/tree-ssa/pr17141-1.c -O2 -fdump-tree-forwprop2 -fno-show-column -S -isystem /tmp/hpautotest-gcc1/cris-elf/gccobj/cris-elf/./newlib/targ-include -isystem /tmp/hpautotest-gcc1/gcc/newlib/libc/include -o pr17141-1.s (timeout = 300)
PASS: gcc.dg/tree-ssa/pr17141-1.c (test for excess errors)
FAIL: gcc.dg/tree-ssa/pr17141-1.c scan-tree-dump-times & 0
and g++.log:
Executing on host: /tmp/hpautotest-gcc1/cris-elf/gccobj/gcc/testsuite/g++/../../g++ -B/tmp/hpautotest-gcc1/cris-elf/gccobj/gcc/testsuite/g++/../../ /tmp/hpautotest-gcc1/gcc/gcc/testsuite/g++.dg/tree-ssa/pr31146.C -nostdinc++ -I/tmp/hpautotest-gcc1/cris-elf/gccobj/cris-elf/libstdc++-v3/include/cris-elf -I/tmp/hpautotest-gcc1/cris-elf/gccobj/cris-elf/libstdc++-v3/include -I/tmp/hpautotest-gcc1/gcc/libstdc++-v3/libsupc++ -I/tmp/hpautotest-gcc1/gcc/libstdc++-v3/include/backward -I/tmp/hpautotest-gcc1/gcc/libstdc++-v3/testsuite/util -fmessage-length=0 -O -fdump-tree-forwprop1 -S -isystem /tmp/hpautotest-gcc1/cris-elf/gccobj/cris-elf/./newlib/targ-include -isystem /tmp/hpautotest-gcc1/gcc/newlib/libc/include -o pr31146.s (timeout = 300)
PASS: g++.dg/tree-ssa/pr31146.C (test for excess errors)
FAIL: g++.dg/tree-ssa/pr31146.C scan-tree-dump i\[j.*\] = 1;
*) Theoretically but very unlikely, another change in
124634:124640 could cause it, but the other non-library changes
are RTL only.
brgds, H-P