On Wed, Jun 09, 2021 at 12:08:39PM -0700, sunil.k.pandey via Gcc-patches wrote:
> On Linux/x86_64,
>
> 374f93da97fb0378453d503f3cfea4d7a923a89c is the first bad commit
> commit 374f93da97fb0378453d503f3cfea4d7a923a89c
> Author: Richard Biener <rguent...@suse.de>
> Date: Wed Jun 9 14:48:35 2021 +0200
>
> tree-optimization/100981 - fix SLP patterns involving reductions
>
> caused
>
> FAIL: libgomp.fortran/pr100981-2.f90 -O0 execution test
> FAIL: libgomp.fortran/pr100981-2.f90 -O1 execution test
> FAIL: libgomp.fortran/pr100981-2.f90 -O2 execution test
> FAIL: libgomp.fortran/pr100981-2.f90 -O3 -g execution test
> FAIL: libgomp.fortran/pr100981-2.f90 -Os execution test
Aren't the dsdotr and dsdoti variables uninitialized?
Shouldn't they be initialized to zero?
Following fixes it for me on i686-linux, but I haven't checked if it
fails with the patch reverted.
2021-06-10 Jakub Jelinek <ja...@redhat.com>
* testsuite/libgomp.fortran/pr100981-2.f90 (cdcdot): Initialize
dsdotr and dsdoti to 0.
--- libgomp/testsuite/libgomp.fortran/pr100981-2.f90.jj 2021-06-09
22:51:44.548834216 +0200
+++ libgomp/testsuite/libgomp.fortran/pr100981-2.f90 2021-06-10
01:08:18.056464950 +0200
@@ -9,6 +9,8 @@ complex function cdcdot(n, cx)
double precision :: dsdotr, dsdoti, dt1, dt3
kx = 1
+ dsdotr = 0
+ dsdoti = 0
do i = 1, n
dt1 = real(cx(kx))
dt3 = aimag(cx(kx))
Jakub
