PASSIONLab / CombBLAS

The Combinatorial BLAS (CombBLAS) is an extensible distributed-memory parallel graph library offering a small but powerful set of linear algebra primitives specifically targeting graph analytics.
Other
64 stars 22 forks source link

Fixed double-free issue with non-square grids. #9

Closed vbharadwaj-bk closed 3 years ago

vbharadwaj-bk commented 3 years ago

When the process grid is non-square., diagWorld is set as the entire communicator world, which leads to a double free when the communicator grid is destroyed. This commit fixes the double-free issue.

This commit also comments out the warning about returning the entire world as the diagonal world for non-square communication grid (every rank produces this warning for a non-square grid, which be annoying. Maybe a better way to do this would be to only have process 0 report this warning? This would need a call to MPI_Comm_Rank, but I just commented it out. Happy to fix this so the warning is no longer silent.