Skip to content
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

portals4: update the coll-portals4 component to be Big Count compliant #12714

Conversation

tkordenbrock
Copy link
Member

The Big Count implementation bumped the coll base struct version to 3.0.0. This PR updates the coll-portals4 component to use coll base 3.0.0, updates function signatures to be Big Count compliant and fixes some compiler warnings.

Copy link
Contributor

@mdosanjh mdosanjh left a comment

Choose a reason for hiding this comment

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

This is a fairly straightforward set of changes to support Big Count (changing 'int's to size_ts and adding casts to void* in function calls to avoid compiler warnings).

@jsquyres
Copy link
Member

@tkordenbrock @mdosanjh Did you want to merge this PR, or are you intentionally waiting for some reason?

@tkordenbrock tkordenbrock merged commit 57f2404 into open-mpi:main Aug 13, 2024
14 checks passed
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