Skip to content

fortran: build MPI_Sizeof() interface in use-mpi-tkr bindings #5101

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

Conversation

ggouaillardet
Copy link
Contributor

whenever possible.

Add the missing OMPI_FORTRAN_BUILD_SIZEOF macro to Fortran
and add a missing dependency.

Thanks Themos Tsikas for reporting this issue.

Refs #5085

Signed-off-by: Gilles Gouaillardet [email protected]

(back-ported from commit 2abeada)

whenever possible.

Add the missing OMPI_FORTRAN_BUILD_SIZEOF macro to Fortran
and add a missing dependency.

Thanks Themos Tsikas for reporting this issue.

Refs open-mpi#5085

Signed-off-by: Gilles Gouaillardet <[email protected]>

(back-ported from commit open-mpi/ompi@2abeada)
@jsquyres
Copy link
Member

@ThemosTsikas Can you confirm that this PR fixes the issue for you with the v3.0.x branch?

@jsquyres jsquyres added the NEWS label Apr 25, 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.

NEWS comment: Fixed MPI_SIZEOF in the "mpi" Fortran module for the NAG compiler.

(applied to the v2.x, v3.0.x, and v3.1.x branches)

Set mpi.lo dependencies outside of AM conditionals.  Per
open-mpi#5085, make mpi.lo depend on
whatever we decide the sizeof source files are (which may be empty, if
this compiler does not support the Right Stuff for MPI_SIZEOF, or it
may be mpi-tkr-sizeof.[h|f90]).

Signed-off-by: Jeff Squyres <[email protected]>

(cherry picked from commit open-mpi/ompi@5bd02f6)
@bwbarrett bwbarrett merged commit a408c59 into open-mpi:v3.0.x Apr 26, 2018
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