Skip to content

MPICommOnly / mpidummy cleanup; no-MPI_Init support

Podhorszki, Norbert requested to merge github/fork/germasch/pr/mpidummy into master

Created by: germasch

This PR gets rid off ADIOSMPICommOnly.h. This probably dates back to the pre-pimpl cxx11 bindings, but isn't useful anymore. Currently, this header gets installed, but only included in the public (Installed) adios2_c_operator.h, which doesn't need MPI_Comm.

This PR also does some cleanup on mpidummy, including fixing a couple of minor bugs and getting rid of MPI_Error_string, which is never used and also doesn't do what the standard says it should be doing.

ETA: This PR now also includes a new level of MPI wrapping in mpiwrap.{h,cpp}. The functions SMPI_Comm_rank() etc behave like the regular MPI counterpart, but they fall back to mpidummy.h in the case where MPI is not initialized, in order to make ADIOS2 more easily usable from serial code even if ADIOS2 itself has been built with MPI.

Merge request reports