Skip to content

binding/abi: update upstream mpi_abi.h - #7937

Merged
hzhou merged 1 commit into
pmodels:mainfrom
dalcinl:sync-abi
Aug 20, 2026
Merged

binding/abi: update upstream mpi_abi.h#7937
hzhou merged 1 commit into
pmodels:mainfrom
dalcinl:sync-abi

Conversation

@dalcinl

@dalcinl dalcinl commented Aug 19, 2026

Copy link
Copy Markdown
Collaborator
  • Define MPI_ABI_Aint/Offset/Count unconditionally from types in stdint.h
  • Define MPI_ERR_LASTCODE as a decimal integer
  • Define MPI_ORDER_C/FORTRAN as decimal integers
  • Mark deprecated APIs with comments

Pull Request Description

Author Checklist

  • Provide Description
    Particularly focus on why, not what. Reference background, issues, test failures, xfail entries, etc.
  • Commits Follow Good Practice
    Commits are self-contained and do not do two things at once.
    Commit message is of the form: module: short description
    Commit message explains what's in the commit.
  • Passes All Tests
    Whitespace checker. Warnings test. Additional tests via comments.
  • Contribution Agreement
    For non-Argonne authors, check contribution agreement.
    If necessary, request an explicit comment from your companies PR approval manager.

@dalcinl

dalcinl commented Aug 19, 2026

Copy link
Copy Markdown
Collaborator Author

mpi4py CI tests running here: https://github.com/mpi4py/mpi4py-testing/actions/runs/32296031541 ✔️

Comment thread src/binding/abi/mpi_abi.h Outdated
Comment thread src/binding/abi/mpi_abi.h
* Typedef MPI_Aint/Offset/Count unconditionally from types in stdint.h
* Define MPI_ERR_LASTCODE as a decimal integer
* Define MPI_ORDER_C/FORTRAN as decimal integers
* Mark deprecated APIs with comments

@hzhou hzhou left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@hzhou

hzhou commented Aug 20, 2026

Copy link
Copy Markdown
Collaborator

test:mpich/ch3/most
test:mpich/custom
config: mpi-abi

✔️

@hzhou
hzhou merged commit 6a79bf6 into pmodels:main Aug 20, 2026
9 checks passed
@dalcinl
dalcinl deleted the sync-abi branch August 25, 2026 12:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants