Re: [Patch, Fortran] Add CO_BROADCAST

2014-09-30 Thread James Greenhalgh
On Tue, Sep 30, 2014 at 11:59:16AM +0100, Dominique d'Humières wrote: > This is what I have committed as r215715: > > Index: gcc/testsuite/ChangeLog > === > --- gcc/testsuite/ChangeLog (revision 215714) > +++ gcc/testsuite/ChangeLog

Re: [Patch, Fortran] Add CO_BROADCAST

2014-09-30 Thread Dominique d'Humières
This is what I have committed as r215715: Index: gcc/testsuite/ChangeLog === --- gcc/testsuite/ChangeLog (revision 215714) +++ gcc/testsuite/ChangeLog (working copy) @@ -1,3 +1,7 @@ +2014-30-09 Dominique d'Humieres + +

Re: [Patch, Fortran] Add CO_BROADCAST

2014-09-29 Thread Dominique d'Humières
Le 29 sept. 2014 à 23:56, Dominique d'Humières a écrit : > Unless there is an objection I plan to commit tomorrow the following patch > with a change log: > > --- ../_clean/gcc/testsuite/gfortran.dg/coarray_collectives_9.f90 > 2014-09-25 12:14:05.0 +0200 > +++ gcc/testsuite/gfortr

Re: [Patch, Fortran] Add CO_BROADCAST

2014-09-29 Thread Tobias Burnus
On Mon, Sep 29, 2014 at 10:17:04AM +0200, Tobias Burnus wrote: > Dominique Dhumieres wrote: > > The failures for the gfortran.dg/coarray_collectives_9.f90 are fixed > > with the following patch: > > Looks good to me. The patch is OK with a ChangLog. Actually, I missed the following part: ... -

Re: [Patch, Fortran] Add CO_BROADCAST

2014-09-29 Thread Tobias Burnus
Dominique Dhumieres wrote: > The failures for the gfortran.dg/coarray_collectives_9.f90 are fixed > with the following patch: Looks good to me. The patch is OK with a ChangLog. Thanks for the patch and sorry for the test fails. Tobias

Re: [Patch, Fortran] Add CO_BROADCAST

2014-09-27 Thread Dominique Dhumieres
The failures for the gfortran.dg/coarray_collectives_9.f90 are fixed with the following patch: --- ../_clean/gcc/testsuite/gfortran.dg/coarray_collectives_9.f90 2014-09-25 12:14:05.0 +0200 +++ gcc/testsuite/gfortran.dg/coarray_collectives_9.f90 2014-09-27 13:03:41.0 +0200

Re: [Patch, Fortran] Add CO_BROADCAST

2014-09-26 Thread Andreas Schwab
Tobias Burnus writes: > diff --git a/gcc/testsuite/gfortran.dg/coarray_collectives_9.f90 > b/gcc/testsuite/gfortran.dg/coarray_collectives_9.f90 > new file mode 100644 > index 000..90c09c5 > --- /dev/null > +++ b/gcc/testsuite/gfortran.dg/coarray_collectives_9.f90 > @@ -0,0 +1,62 @@ > +! { d

Re: [Patch, Fortran] Add CO_BROADCAST

2014-09-25 Thread Alessandro Fanfarillo
Hi, I'm implementing the co_broadcast on libcafmpi right now. 2014-09-25 8:08 GMT+02:00 Tobias Burnus : > Hi Paul, > > Paul Richard Thomas wrote: >> >> In the check.c error messages, you use 'A argument'. Should you not >> use 'SOURCE argument', following CO BROADCAST (SOURCE, SOURCE IMAGE [, >>

Re: [Patch, Fortran] Add CO_BROADCAST

2014-09-24 Thread Tobias Burnus
Hi Paul, Paul Richard Thomas wrote: In the check.c error messages, you use 'A argument'. Should you not use 'SOURCE argument', following CO BROADCAST (SOURCE, SOURCE IMAGE [, STAT, ERRMSG]) ? I am looking at WG5/N1983 - is there some more recent proposal? Looking at N2027, I see "A, SOURCE_IM

Re: [Patch, Fortran] Add CO_BROADCAST

2014-09-24 Thread Paul Richard Thomas
Hi Tobias, In the check.c error messages, you use 'A argument'. Should you not use 'SOURCE argument', following CO BROADCAST (SOURCE, SOURCE IMAGE [, STAT, ERRMSG]) ? I am looking at WG5/N1983 - is there some more recent proposal? When do you intend to implement a _gfortran_caf_co_broadcast that

[Patch, Fortran] Add CO_BROADCAST

2014-09-20 Thread Tobias Burnus
This patch adds a CO_BROADCAST and prepares a bit for CO_REDUCE. Both functions permit arguments with allocatable components (nonpolymophic or polymorphic), CO_BROADCAST also permits polymorphic arguments. This patch doesn't support allocatable/polymorphic arguments but otherwise CO_BROADCAST