2015-01-02 17:24 GMT+01:00 Tobias Burnus <bur...@net-b.de>: >>> Looking through the stashed patches, I realized a pending (unsubmitted) >>> patch, showing that both a test case for CO_REDUCE was missing and that >>> libcaf_single didn't include co_reduce. >>> >>> This patch adds them. >>> >>> Build and regtested on x86-64-gnu-linux. >>> OK for the trunk? >> >> looks rather trivial to me. All the correctness checking for CO_REDUCE >> is already there, right? > > Do you mean compile-time diagnostic for invalid arguments?
Yeah, that's what I meant. > Yes, those are at > gfortran.dg/coarray_collectives_9.f90 to > gfortran.dg/coarray_collectives_16.f90. Yes, just found those. > For run time, this patch adds a check, albeit it is not really checkable > with num_images() == 1. You mean via the STAT and ERRMSG arguments? In any case, the patch is ok from my side. Cheers, Janus