There were a few instances where the MPI-1 error handling routines were used. Everywhere else, the default MPI crash occurred. Now all MPI crashes will report more information through this error handler.
Awesome, thanks for fixing this. I can conform that Enzo builds with this change under openmpi 4.0.0.
Thanks John – this looks good. I wonder if the CommunicationErrorHandlerFn() function itself shouldn’t go in CommunicationInitialize.C rather than CommunicationNonblockingRoutines.C since it is now the standard MPI error handler in use throughout?