-
Notifications
You must be signed in to change notification settings - Fork 62
MPI_CXX_BOOL not defined in Microsoft MPI #164
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
Comments
I'll test it with implementations I have (2 flavors of Mpich and OpenMPI) and merge it. |
@aminiussi Thanks for the reply. I don't use Boost.MPI myself, I just try to compile it in the VCPKG package manager (microsoft/vcpkg#44833). After reverting the changes from commit 33765a1 it works on Windows. As a quick and easy solution I would suggest using something like: #ifdef MPI_CXX_BOOL
// use MPI_CXX_BOOL
#else
// use the previous implementation for bool based on MPI_BYTE
#endif As a long term solution set up CI with the MSVC compiler as done for other Boost libraries. |
The issue with
is that I'm not sure Maybe something similar can be done with MSMPI_VER (https://github.com/microsoft/Microsoft-MPI/blob/master/src/include/mpi.h) but I don't know where/how the value is set. Since there seems to be no |
When building with the MSVC compiler and Microsoft MPI, the following error occurs:
MPI_CXX_BOOL
is not defined in Microsoft MPI interface: https://learn.microsoft.com/en-us/message-passing-interface/mpi-datatype-enumerationThe problem has been introduced in commit 33765a1.
Would it be possible to address this by using
MPI_C_BOOL
as an alternative for a boolean datatype? This might resolve the compatibility issue with Microsoft MPI.@aminiussi Could you please look into this and provide guidance?
The text was updated successfully, but these errors were encountered: