Skip to content

v4.1.x: Add Missing MPI_F_XXX C constants #10530 #10534

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
merged 2 commits into from
Jun 30, 2022
Merged

v4.1.x: Add Missing MPI_F_XXX C constants #10530 #10534

merged 2 commits into from
Jun 30, 2022

Conversation

drwootton
Copy link
Contributor

This is a cherry pick of #10491 to include MPI Fortran constants used by C code in mpi.in.h for the v4.1.x branch.
This also includes commit d44ec62 for #10528

Signed-off-by: David Wootton dwootton@us.ibm.com

Signed-off-by: David Wootton <dwootton@us.ibm.com>
(cherry picked from commit f27c1de)
@drwootton drwootton requested review from jsquyres and jjhursey June 29, 2022 20:41
@drwootton drwootton self-assigned this Jun 29, 2022
@jjhursey jjhursey added this to the v4.1.5 milestone Jun 29, 2022
@jsquyres
Copy link
Member

Same as #10533: it's odd that the authorship of the commit changed, and that it contains 2 cherry-pick notices, not one.

Add some clarifying comments, update some stale comments, and fix some
whitespace.

No code or logic changes.

Signed-off-by: Jeff Squyres <jsquyres@cisco.com>
(cherry picked from commit d5f871d)
@jjhursey
Copy link
Member

Looks like the wrong hash was grabbed (the v5 PR hash instead of the 'main' hash). It's all fixed up now.

@jsquyres
Copy link
Member

bot:aws:retest

@jprotze
Copy link

jprotze commented Aug 8, 2023

This PR left OMPI_F_STATUS_SIZE undefined.

The macro definition was introduced in main by:
7fce2f3

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.

6 participants