[Bug fortran/83064] DO CONCURRENT inconsistent results

2018-04-08 Thread tkoenig at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83064 Thomas Koenig changed: What|Removed |Added CC||tkoenig at gcc dot gnu.org --- Comment #

[Bug fortran/83064] DO CONCURRENT inconsistent results

2017-11-26 Thread cfztol at hotmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83064 --- Comment #14 from Christian Felter --- I looked into the working draft of Fortran 2015 (J3/16-007r1). In Note 12.52 it says: The above constraints are designed to guarantee that a pure procedure is free from side effects (modifications of dat

[Bug fortran/83064] DO CONCURRENT inconsistent results

2017-11-24 Thread dominiq at lps dot ens.fr
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83064 --- Comment #13 from Dominique d'Humieres --- > > I confirm that using > > > > pik = compute( low(i), high(i) ) > > pi(i) = sum(pik) > > > > gives the right result. > > > > Does it means that the 'sum' in 'sum(compute( low(i), hig

[Bug fortran/83064] DO CONCURRENT inconsistent results

2017-11-24 Thread rguenther at suse dot de
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83064 --- Comment #12 from rguenther at suse dot de --- On Fri, 24 Nov 2017, jakub at gcc dot gnu.org wrote: > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83064 > > --- Comment #11 from Jakub Jelinek --- > There should be no duplication except for

[Bug fortran/83064] DO CONCURRENT inconsistent results

2017-11-24 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83064 --- Comment #11 from Jakub Jelinek --- There should be no duplication except for vars mentioned in the clauses on OpenMP constructs (explicit or implicit). So, vars declared outside of the region and referenced inside of the region are either im

[Bug fortran/83064] DO CONCURRENT inconsistent results

2017-11-24 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83064 Richard Biener changed: What|Removed |Added CC||jakub at gcc dot gnu.org --- Comment #1

[Bug fortran/83064] DO CONCURRENT inconsistent results

2017-11-24 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83064 --- Comment #9 from Richard Biener --- How does this work with OpenMP and compiler/FE generated temporary arrays? If I just do !$OMP parallel do do i = 1,nsplit pi(i) = sum(compute( low(i), high(i) )) end do I still get #prag

[Bug fortran/83064] DO CONCURRENT inconsistent results

2017-11-24 Thread rguenther at suse dot de
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83064 --- Comment #8 from rguenther at suse dot de --- On Thu, 23 Nov 2017, dominiq at lps dot ens.fr wrote: > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83064 > > --- Comment #7 from Dominique d'Humieres --- > > I looked at the IL from the Fortra

[Bug fortran/83064] DO CONCURRENT inconsistent results

2017-11-23 Thread dominiq at lps dot ens.fr
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83064 --- Comment #7 from Dominique d'Humieres --- > I looked at the IL from the Fortran FE and it clearly uses a single memory > area for tmp for each outer loop iteration. That is, the memory is allocated > by the caller. I confirm that using

[Bug fortran/83064] DO CONCURRENT inconsistent results

2017-11-23 Thread rguenther at suse dot de
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83064 --- Comment #6 from rguenther at suse dot de --- On November 23, 2017 6:30:34 PM GMT+01:00, "dominiq at lps dot ens.fr" wrote: >https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83064 > >--- Comment #5 from Dominique d'Humieres >--- >> As I said in

[Bug fortran/83064] DO CONCURRENT inconsistent results

2017-11-23 Thread dominiq at lps dot ens.fr
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83064 --- Comment #5 from Dominique d'Humieres --- > As I said in the other bug we're seeing > >real(real64), dimension(nsplit) :: tmp > > as "shared" amongst the outer loop iterations. > > ... > > Not sure if you need to mark compute somehow t

[Bug fortran/83064] DO CONCURRENT inconsistent results

2017-11-23 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83064 Richard Biener changed: What|Removed |Added Keywords||wrong-code --- Comment #4 from Richard

[Bug fortran/83064] DO CONCURRENT inconsistent results

2017-11-22 Thread dominiq at lps dot ens.fr
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83064 Dominique d'Humieres changed: What|Removed |Added CC||rguenth at gcc dot gnu.org --- Co

[Bug fortran/83064] DO CONCURRENT inconsistent results

2017-11-21 Thread cfztol at hotmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83064 --- Comment #2 from Christian Felter --- I did "svn up" and after that I found the revision number like this: $ svn log --limit 1 r254890 | hjl | 2017-11-17 19:37:58 +0100

[Bug fortran/83064] DO CONCURRENT inconsistent results

2017-11-21 Thread dominiq at lps dot ens.fr
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83064 Dominique d'Humieres changed: What|Removed |Added Status|UNCONFIRMED |NEW Last reconfirmed|