Merge pull request GraphBLAS#300 from GraphBLAS/with_GraphBLAS_v10

fix leak in test_Coarsen_matching; add more to "make memcheck"

Author: DrTimothyAldenDavis

Makefile

@@ -76,7 +76,12 @@ test: library
 verbose_test: library
 	( cd build && ctest . --verbose || ctest . --rerun-failed --output-on-failure )
 
-memcheck: debug
+# memcheck: compile with -g and no OpenMP (the latter gives spurious leaks from
+# dlopen and dlinit), and run the tests with valgrind.  For best results,
+# compile GraphBLAS without OpenMP and with the JIT disabled.
+memcheck: distclean
+	( cd build && cmake $(CMAKE_OPTIONS) -DCMAKE_BUILD_TYPE=Debug -DLAGRAPH_USE_OPENMP=0 .. )
+	( cd build && cmake --build . --config Release -j${JOBS} )
 	( cd build && ctest . -T memcheck )
 
 # target used in CI

experimental/test/test_Coarsen_Matching.c

@@ -132,6 +132,7 @@ void test_Coarsen_Matching () {
         OK (LAGraph_Free ((void**)(&rows), msg)) ;
         OK (LAGraph_Free ((void**)(&cols), msg)) ;
         OK (LAGraph_Free ((void**)(&vals), msg)) ;
+        OK (GrB_wait (A, GrB_MATERIALIZE)) ;
 
         // ================== graph generation done ======================================
 
@@ -163,6 +164,7 @@ void test_Coarsen_Matching () {
                 GrB_Matrix A_fp32 = NULL ;
                 OK (GrB_Matrix_new (&A_fp32, GrB_FP32, n, n)) ;
                 OK (GrB_assign (A_fp32, NULL, NULL, G->A, GrB_ALL, n, GrB_ALL, n, NULL)) ;
+                OK (GrB_free (&(G->A))) ;
                 OK (GrB_free (&(G->AT))) ;
                 G->A = A_fp32 ;
                 A_fp32 = NULL ;