sys comm: remove redundant self-call to MPI_Attr_get in DelComm_Outer
MS-MPI (2008 and 2012) invalidates the communicator before calling the
attribute destructor. This is considered a bug as evidenced by test
attrdeleteget added to MPICH in:
Author: Jed Brown <email@example.com>
Date: Sat May 18 12:55:38 2013 -0500
test: add attrdeleteget, MPI_Attr_get called from delete_fn
With MS-MPI 64-bit from HPC Pack 2008 and 2012, MPI_Attr_get returns
error code 773 when called from delete_fn on a communicator obtained
The standard is not explicit that the 'comm' argument of delete_fn must
be valid, so this test is only in effect when !USE_STRICT_MPI.
Signed-off-by: Jed Brown <firstname.lastname@example.org>
Signed-off-by: Satish Balay <email@example.com>
However, in PETSc, the attribute value is already passed as an argument,
so we need not call MPI_Attr_get to obtain it.
Note that although we still call MPI_Attr_delete and MPI_Attr_get on the
inner comm, that communicator has not been invalidated yet.