BerkeleyLab / caffeine

A parallel runtime library for Fortran compilers
https://berkeleylab.github.io/caffeine/
Other
41 stars 7 forks source link

Add optional stat arguments to co_sum, co_min, & co_max #10

Closed rouson closed 2 years ago

ktras commented 2 years ago

@rouson In caf_co_max_test.f90 you did not assign the variable status a value before passing it to the collective, whereas in the other tests, you did so. Did you mean for caf_co_max_test.f90 to differ like that? If that was your intent, then this looks good to me.

rouson commented 2 years ago

@ktras great catch. I meant to initialize it to ensure that it gets redefined to 0 upon exiting each invoked C function.

rouson commented 2 years ago

I just pushed a fix and will merge after tests pass.