Skip to content

Remove C-wrapper for MPI_BCAST #62

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 4 commits into from
Mar 31, 2025
Merged

Remove C-wrapper for MPI_BCAST #62

merged 4 commits into from
Mar 31, 2025

Conversation

adit4443ya
Copy link
Collaborator

Towards #21

@adit4443ya
Test: Add MPI_Bcast
2f594e7
@adit4443ya
Use unified c_mpi_bcast for all datatypes
53ff8a0 
- This is due to the fact that we would transfer the type(C_PTR) which will corresponds to the void* in the MPI_Bcast args -Also create C-binding wrapper for get_c_datatype_from_fortran
@adit4443ya
Remove C-wrapper for MPI_BCAST
df811e8
@adit4443ya
Copy link
Collaborator Author

This test fails only with lfortran
need a fix
will make a MRE or i for now i can open issue in this repo to keep it remembered

@adit4443ya
Copy link
Collaborator Author

Here made it #63

@adit4443ya
Copy link
Collaborator Author

Now testing with just simpler MRE which broadcasts an integer.

@adit4443ya adit4443ya requested a review from gxyd March 31, 2025 10:10
@gxyd gxyd merged commit c98e71f into lfortran:main Mar 31, 2025
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gxyd gxyd gxyd approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@adit4443ya @gxyd