Fix warnings ready for release (#273)

* Fix incorrectly named header for AMGX_matrix_check_symmetry.
* Fixing warnings ready for release.

Author: mattmartineau

include/amgx_c.h

@@ -608,7 +608,7 @@ AMGX_RC AMGX_API AMGX_matrix_upload_all_global_32
  const void *diag_data,
  AMGX_distribution_handle distribution);
 
-AMGX_RC AMGX_API AMGX_check_matrix_symmetry
+AMGX_RC AMGX_API AMGX_matrix_check_symmetry
 (AMGX_matrix_handle mtx,
  int* structurally_symmetric,
  int* symmetric);

src/amgx_cusparse.cu

@@ -1209,18 +1209,18 @@ inline void Cusparse::bsrmv( cusparseHandle_t handle, cusparseDirection_t dir, c
                              double *y,
                              const cudaStream_t& stream)
 {
-    // Run cuSparse on selected stream
-    cusparseSetStream(handle, stream);
-
     #ifndef DISABLE_MIXED_PRECISION
+        // Run cuSparse on selected stream
+        cusparseSetStream(handle, stream);
+
         const double *d_bsrVal = reinterpret_cast<const double *>(const_cast<float *>(bsrVal)); // this works due to private API call in the matrix initialization which sets cusparse matrix description in the half precision mode
         cusparseCheckError(cusparseDbsrxmv(handle, dir, trans, mb, mb, nb, nnzb, alpha, descr, d_bsrVal, bsrMaskPtr, bsrRowPtr, bsrRowPtr + 1, bsrColInd, blockDim, x, beta, y));
+
+        // Reset cuSparse to default stream
+        cusparseSetStream(handle, 0);
     #else
         FatalError("Mixed precision modes not currently supported for CUDA 10.1 or later.", AMGX_ERR_NOT_IMPLEMENTED);
     #endif
-
-    // Reset cuSparse to default stream
-    cusparseSetStream(handle, 0);
 }
 
 // Custom implementation of matrix-vector product to replace the original bsrxmv,
@@ -1491,18 +1491,18 @@ inline void Cusparse::bsrmv( cusparseHandle_t handle, cusparseDirection_t dir, c
                              cuDoubleComplex *y,
                              const cudaStream_t& stream)
 {
-    // Run cuSparse on selected stream
-    cusparseSetStream(handle, stream);
-
     #ifndef DISABLE_MIXED_PRECISION
+        // Run cuSparse on selected stream
+        cusparseSetStream(handle, stream);
+
         const cuDoubleComplex *d_bsrVal = reinterpret_cast<cuDoubleComplex *>(const_cast<cuComplex *>(bsrVal));
         cusparseCheckError(cusparseZbsrxmv(handle, dir, trans, mb, mb, nb, nnzb, alpha, descr, d_bsrVal, bsrMaskPtr, bsrRowPtr, bsrRowPtr + 1, bsrColInd, blockDim, x, beta, y));
+
+        // Reset cuSparse to default stream
+        cusparseSetStream(handle, 0);
     #else
         FatalError("Mixed precision modes not currently supported for CUDA 10.1 or later.", AMGX_ERR_NOT_IMPLEMENTED);
     #endif
-
-    // Reset cuSparse to default stream
-    cusparseSetStream(handle, 0);
 }
 
 

src/distributed/distributed_manager.cu

@@ -1901,7 +1901,6 @@ void *DistributedManagerBase<TConfig>::getHostPointerForData(void *ptr, size_t s
         //you are in case 1 or 3 from the above comment
         ptr_h = ptr;
     }
-#endif
 
     cudaGetLastError(); //to reset last error
 
@@ -1912,6 +1911,7 @@ void *DistributedManagerBase<TConfig>::getHostPointerForData(void *ptr, size_t s
     }
 
     return ptr_h;
+#endif
 }
 
 // if pointer is host pointer - returns data. If it is device pointer - copies it to the m_pinned_buffer and returns pointer to m_pinned_buffer

src/solvers/gauss_seidel_solver.cu

@@ -158,7 +158,7 @@ void GaussSeidelSolver<TemplateConfig<AMGX_host, t_vecPrec, t_matPrec, t_indPrec
         {
             if (A.col_indices[j] == i)
             {
-                this->diag[i] = /*static_cast<ValueTypeB>(1) /*/ A.values[j];
+                this->diag[i] = A.values[j];
                 break;
             }