--- Comment #39 from spop at gcc dot gnu dot org 2009-01-14 21:11 ---
Subject: Bug 38431
Author: spop
Date: Wed Jan 14 21:11:05 2009
New Revision: 143384
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=143384
Log:
2009-01-14 Sebastian Pop
PR middle-end/38431
*
--- Comment #38 from spop at gcc dot gnu dot org 2009-01-14 14:36 ---
Fixed.
--
spop at gcc dot gnu dot org changed:
What|Removed |Added
Status|NEW
--- Comment #37 from spop at gcc dot gnu dot org 2009-01-14 14:35 ---
Subject: Bug 38431
Author: spop
Date: Wed Jan 14 14:35:27 2009
New Revision: 143372
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=143372
Log:
2009-01-14 Sebastian Pop
PR middle-end/38431
*
--- Comment #36 from jv244 at cam dot ac dot uk 2009-01-14 12:08 ---
(In reply to comment #35)
> (In reply to comment #33)
> > Attached a fix for this PR. I will regstrap and submit for review.
>
> while I'll apply it to current trunk, retest CP2K, and update this PR with the
> results
--- Comment #35 from jv244 at cam dot ac dot uk 2009-01-14 10:51 ---
(In reply to comment #33)
> Attached a fix for this PR. I will regstrap and submit for review.
while I'll apply it to current trunk, retest CP2K, and update this PR with the
results.
Thanks,
Joost
--
http://gcc
--- Comment #33 from sebpop at gmail dot com 2009-01-14 10:20 ---
Subject: Re: [graphite] several ICEs with CP2K (summary)
Attached a fix for this PR. I will regstrap and submit for review.
Sebastian
--- Comment #34 from sebpop at gmail dot com 2009-01-14 10:20 ---
Created
Attached a fix for this PR. I will regstrap and submit for review.
Sebastian
2009-01-14 Sebastian Pop
PR middle-end/38431
* graphite.c (get_vdef_before_scop, scop_adjust_vphi): New.
(scop_adjust_phis_for_liveouts): Call scop_adjust_vphi.
(gloog): Do not call cleanup_tree_cfg.
(graphite_t
--- Comment #32 from jv244 at cam dot ac dot uk 2009-01-14 06:49 ---
(In reply to comment #30)
> Subject: Re: [graphite] several ICEs with CP2K (summary)
>
> Thanks for the clarification, I managed to reproduce the fail. The
> problem comes from the fact that we do not generate code f
--- Comment #30 from sebpop at gmail dot com 2009-01-13 21:57 ---
Subject: Re: [graphite] several ICEs with CP2K (summary)
Thanks for the clarification, I managed to reproduce the fail. The
problem comes from the fact that we do not generate code for a scalar
reduction that is not det
Thanks for the clarification, I managed to reproduce the fail. The
problem comes from the fact that we do not generate code for a scalar
reduction that is not detected as a scalar reduction with the variable
connection$dim$1$lbound.
In the attached output from debug_loops (3) I selected the regio
--- Comment #29 from jv244 at cam dot ac dot uk 2009-01-13 20:33 ---
(In reply to comment #28)
> the graphite branch. However I'm not able to run the test that you
> reported failing:
>
> ./cp2k.sopt canonical.inp
> CP2K: The specified file canonical.inp can not be opened, it does not
--- Comment #28 from sebpop at gmail dot com 2009-01-13 19:52 ---
Subject: Re: [graphite] several ICEs with CP2K (summary)
Hi,
I compiled BLAS and LAPACK with the gfortran compiler of the graphite
branch such that I could test the CP2K benchmark. On my laptop, that
is an amd64-linux,
Hi,
I compiled BLAS and LAPACK with the gfortran compiler of the graphite
branch such that I could test the CP2K benchmark. On my laptop, that
is an amd64-linux, "make test" passes with the gfortran compiler from
the graphite branch. However I'm not able to run the test that you
reported failing
--- Comment #27 from sebpop at gmail dot com 2009-01-11 13:42 ---
Subject: Re: [graphite] several ICEs with CP2K (summary)
On Sun, Jan 11, 2009 at 6:58 AM, jv244 at cam dot ac dot uk
wrote:
>> I'll see if I
>> can narrow down the problem to the single subroutine (change_bond_length)
--- Comment #26 from jv244 at cam dot ac dot uk 2009-01-11 12:58 ---
(In reply to comment #25)
> I'll see if I
> can narrow down the problem to the single subroutine (change_bond_length)
> which
> I suspect is the issue. [all of this with trunk 143207]
yes, just looking at change_bond_
--- Comment #25 from jv244 at cam dot ac dot uk 2009-01-11 12:30 ---
(In reply to comment #22)
> Program received signal SIGSEGV, Segmentation fault.
> __mc_moves_MOD_change_bond_length () at
> /scratch/vondele/clean/cp2k/src/../src/mc_moves.F:1434
> 1434 atom_b(:)=0
>
I had a
--- Comment #24 from jv244 at cam dot ac dot uk 2009-01-09 06:18 ---
(In reply to comment #22)
> Program received signal SIGSEGV, Segmentation fault.
> __mc_moves_MOD_change_bond_length () at
> /scratch/vondele/clean/cp2k/src/../src/mc_moves.F:1434
> 1434 atom_b(:)=0
>
additio
--- Comment #23 from jv244 at cam dot ac dot uk 2009-01-09 06:16 ---
Created an attachment (id=17062)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=17062&action=view)
additional cp2k input
With this CP2K testcase the segfault can be reproduced. After building cp2k,
just run:
cp2k
--- Comment #22 from jv244 at cam dot ac dot uk 2009-01-09 05:38 ---
(In reply to comment #21)
> Thanks for testing. Can you close the bug after the CP2K testsuite passes?
unfortunately, there is still one runtime segfault, I have no reduced testcase
so far, but this is the backtrace:
--- Comment #21 from sebpop at gmail dot com 2009-01-08 19:53 ---
Subject: Re: [graphite] several ICEs with CP2K (summary)
> the testcase provide runs fine (AFAICT) with current trunk. I'll run the full
> CP2K testsuite to test somewhat better.
Thanks for testing. Can you close the b
--- Comment #20 from jv244 at cam dot ac dot uk 2009-01-08 19:30 ---
(In reply to comment #19)
> Could you try to run again the make
> test and report the status of CP2K?
Hi Sebastian,
the testcase provide runs fine (AFAICT) with current trunk. I'll run the full
CP2K testsuite to test
--- Comment #19 from spop at gcc dot gnu dot org 2009-01-08 17:10 ---
Hi,
With the latest trunk CP2K compiles with -O2 -floop-block. I tried to
run CP2K with -floop-block but I got an error due to the libblas
installation on my system (missing symbols in libblas that uses the
libgfortr
--- Comment #18 from jv244 at cam dot ac dot uk 2009-01-07 20:52 ---
(In reply to comment #17)
> Thanks for the update. I suspect that this is due to -floop-block.
> There are two more bugs 38559 and 38499 that we're looking at for
> fixing -floop-block.
yes, I was able to derive a sma
--- Comment #17 from sebpop at gmail dot com 2009-01-07 19:23 ---
Subject: Re: [graphite] several ICEs with CP2K (summary)
> I checked that current trunk (i.e. not graphite branch) still generates a
> segfaulting executable with
>
> FCFLAGS = -g -O2 -ffast-math -funroll-loops -ftree-ve
--- Comment #16 from jv244 at cam dot ac dot uk 2009-01-07 19:07 ---
I checked that current trunk (i.e. not graphite branch) still generates a
segfaulting executable with
FCFLAGS = -g -O2 -ffast-math -funroll-loops -ftree-vectorize -march=native
-ffree-form -fgraphite -fgraphite-identit
--- Comment #15 from jv244 at cam dot ac dot uk 2008-12-15 19:44 ---
(In reply to comment #14)
> OK, full code compiles & runs the test example with -fgraphite
> -fgraphite-identity -fno-strict-overflow
compiling with
-g -O2 -ffast-math -funroll-loops -ftree-vectorize -march=native -f
--- Comment #14 from jv244 at cam dot ac dot uk 2008-12-15 19:06 ---
(In reply to comment #13)
>
> (i.e. no need for -fno-strict-aliasing) I'll give it a try on the full code.
OK, full code compiles & runs the test example with -fgraphite
-fgraphite-identity -fno-strict-overflow
--
--- Comment #13 from jv244 at cam dot ac dot uk 2008-12-15 17:31 ---
(In reply to comment #12)
> Could you try with the addition of " -fno-strict-aliasing
> -fno-strict-overflow"? See pr38520.
The testcase in PR38492 indeed works if I use:
gfortran -O2 -ffast-math -funroll-loops -ftree
--- Comment #12 from dominiq at lps dot ens dot fr 2008-12-14 15:54 ---
Could you try with the addition of " -fno-strict-aliasing
-fno-strict-overflow"? See pr38520.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38431
--- Comment #11 from jv244 at cam dot ac dot uk 2008-12-13 08:39 ---
(In reply to comment #10)
> (In reply to comment #9)
> > Unfortunately, '-fgraphite -fgraphite-identity -floop-block
> > -floop-strip-mine
> > -floop-interchang' goes generate an executable, but it is miscompiled and
>
--- Comment #10 from jv244 at cam dot ac dot uk 2008-12-11 21:41 ---
(In reply to comment #9)
> Unfortunately, '-fgraphite -fgraphite-identity -floop-block -floop-strip-mine
> -floop-interchang' goes generate an executable, but it is miscompiled and
> segfaults.
reduced testcase in PR38
--- Comment #9 from jv244 at cam dot ac dot uk 2008-12-11 20:17 ---
Unfortunately, '-fgraphite -fgraphite-identity -floop-block -floop-strip-mine
-floop-interchang' goes generate an executable, but it is miscompiled and
segfaults.
It can be reproduced with the testcase in the initial co
--- Comment #8 from jv244 at cam dot ac dot uk 2008-12-11 19:19 ---
(In reply to comment #7)
> with current graphite, i see 4 files failing with -fgraphite
> -fgraphite-identity:
>
> lebedev.F, colvar_types.F, qs_linres_nmr_shift.F, constraint_clv.F
>
all fixed with current graphite b
--- Comment #7 from jv244 at cam dot ac dot uk 2008-12-10 22:33 ---
with current graphite, i see 4 files failing with -fgraphite
-fgraphite-identity:
lebedev.F, colvar_types.F, qs_linres_nmr_shift.F, constraint_clv.F
I'm assuming that these are all incarnations of PR38461, but can look
--- Comment #6 from jv244 at cam dot ac dot uk 2008-12-09 21:11 ---
ps_wavelet_util.F is now PR38463
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38431
--- Comment #5 from jv244 at cam dot ac dot uk 2008-12-09 21:02 ---
lebedev.F on PR38461
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38431
--- Comment #4 from grosser at gcc dot gnu dot org 2008-12-09 20:24 ---
mltfftsg.F fails on Bug38459
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38431
--- Comment #3 from grosser at gcc dot gnu dot org 2008-12-09 20:10 ---
Thanks for these test cases.
My commit fixed at least one failure, but there are more.
To help us it would be great to get a little bit structure in the failures.
Just get for every failing test a backtrace and cre
--- Comment #2 from jv244 at cam dot ac dot uk 2008-12-09 19:41 ---
This is a simple testcase for one of the first segfaults, observed with current
graphite branch:
gfortran -c -O2 -ffree-form -fgraphite -fgraphite-identity test.f90
test.f90: In function matmov:
test.f90:1: internal c
--- Comment #1 from jv244 at cam dot ac dot uk 2008-12-09 08:42 ---
Tobias,
you might be interested to check if your recent patch also fixed these bugs.
Otherwise I should be able to give it a round of testing before the weekend.
--
jv244 at cam dot ac dot uk changed:
W
40 matches
Mail list logo
