Skip to content

Conversation

@kawashima-fj
Copy link
Member

This is v4.0.x version of #5799.

@jsquyres Please review.

PR for v3.0.x will follow after #5781 is merged.


Following the commit f750c69, I compared
ompi/mpi/fortran/use-mpi-f08/*.F90 and
ompi/mpi/fortran/use-mpi-f08/profile/p*.F90, and
ompi/mpi/fortran/use-mpi-f08/mod/mpi-f08-interfaces.F90 and
ompi/mpi/fortran/use-mpi-f08/mod/pmpi-f08-interfaces.F90.

There are many differences. Some are bugs of MPI_*, some are bugs of PMPI_*. I'm not sure how these bugs affect applications.

To make it easy to compare these files future, I also removed editorial differences.

(cherry picked from commit cf6d28c)

Following the commit f750c69, I compared
`ompi/mpi/fortran/use-mpi-f08/*.F90` and
`ompi/mpi/fortran/use-mpi-f08/profile/p*.F90`, and
`ompi/mpi/fortran/use-mpi-f08/mod/mpi-f08-interfaces.F90` and
`ompi/mpi/fortran/use-mpi-f08/mod/pmpi-f08-interfaces.F90`.

There are many differences. Some are bugs of `MPI_*`, some are
bugs of `PMPI_*`. I'm not sure how these bugs affect applications.

To make it easy to compare these files future, I also removed
editorial differences.

Signed-off-by: KAWASHIMA Takahiro <t-kawashima@jp.fujitsu.com>
(cherry picked from commit cf6d28c)
@kawashima-fj kawashima-fj added this to the v4.0.0 milestone Oct 5, 2018
@kawashima-fj kawashima-fj requested a review from jsquyres October 5, 2018 00:31
@kawashima-fj kawashima-fj changed the title fortran/use-mpi-f08: Correct f08 routine signatures v4.0.x: fortran/use-mpi-f08: Correct f08 routine signatures Oct 5, 2018
Copy link
Member

@jsquyres jsquyres left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There are spurious whitespace changes in here (which I think are intentional), but it's otherwise ok.

@hppritcha hppritcha merged commit d18ea98 into open-mpi:v4.0.x Oct 9, 2018
@kawashima-fj kawashima-fj deleted the pr/v4.0.x/correct-f08-signatures branch October 10, 2018 02:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants