Merge pull request #1203 from LLNL/atkins12/remove-petsc-warnings

Remove excess perfomance warnings in PETSc solver wrappers
LLNL · Aug 9, 2024 · 19476f2 · 19476f2
2 parents f84f013 + 606a4d6
commit 19476f2
Showing 1 changed file with 2 additions and 4 deletions.
diff --git a/src/serac/numerics/petsc_solvers.cpp b/src/serac/numerics/petsc_solvers.cpp
@@ -598,7 +598,7 @@ PetscErrorCode convertKSPPreSolve(KSP ksp, [[maybe_unused]] Vec rhs, [[maybe_unu
   if (!solver->checked_for_convert_ || solver->needs_hypre_wrapping_) {
     PetscBool is_hypre;
     PetscCall(PetscObjectTypeCompare(reinterpret_cast<PetscObject>(A), MATHYPRE, &is_hypre));
-    SLIC_WARNING_ROOT_IF(
+    SLIC_DEBUG_ROOT_IF(
         is_hypre && petsc_pc,
         "convertKSPPreSolve(...) - MATHYPRE is not supported for most PETSc preconditioners, converting to MATAIJ.");
     if (!is_hypre || petsc_pc) {
@@ -703,9 +703,7 @@ void PetscKSPSolver::SetOperator(const mfem::Operator& op)
         SLIC_DEBUG_ROOT("PetscKSPSolver::SetOperator(...) - Wrapping existing HYPRE matrix");
         pA = new mfem::PetscParMatrix(hA, wrap_ ? PETSC_MATSHELL : PETSC_MATHYPRE);
       } else {
-        SLIC_WARNING_ROOT(
-            "PetscKSPSolver::SetOperator(...) - Converting operator, consider using PetscParMatrix to avoid "
-            "conversion costs");
+        SLIC_DEBUG_ROOT("PetscKSPSolver::SetOperator(...) - Converting operator from HYPRE to MATAIJ");
         pA = new mfem::PetscParMatrix(hA, wrap_ ? PETSC_MATSHELL : PETSC_MATAIJ);
       }
     } else if (oA) {