Skip to content

Conversation

@kawashima-fj
Copy link
Member

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

@jsquyres Please review.

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.

(back-ported 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>
(back-ported from commit cf6d28c)
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 to make the PMPI/MPI interfaces identical), but it's otherwise ok.

@bwbarrett bwbarrett merged commit d656500 into open-mpi:v3.0.x Oct 9, 2018
@kawashima-fj kawashima-fj deleted the pr/v3.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