add free to fix memory leak

AMReX-Combustion · Feb 12, 2025 · 3a45c2d · 3a45c2d
1 parent 574b74e
commit 3a45c2d
Show file tree

Hide file tree

Showing 4 changed files with 17 additions and 1 deletion.
diff --git a/Exec/RegTests/HITDecay/pelelmex_prob.cpp b/Exec/RegTests/HITDecay/pelelmex_prob.cpp
@@ -196,3 +196,14 @@ PeleLM::readProbParm() // NOLINT(readability-make-member-function-const)
     amrex::Gpu::hostToDevice, winput.begin(), winput.end(),
     PeleLM::prob_parm->d_winput);
 }
+
+void
+PeleLM::freeProbParm()
+{
+  amrex::Print() << "Freeing prob parm" << std::endl;
+  amrex::The_Arena()->free(PeleLM::prob_parm->d_xarray);
+  amrex::The_Arena()->free(PeleLM::prob_parm->d_xdiff);
+  amrex::The_Arena()->free(PeleLM::prob_parm->d_uinput);
+  amrex::The_Arena()->free(PeleLM::prob_parm->d_vinput);
+  amrex::The_Arena()->free(PeleLM::prob_parm->d_winput);
+}
diff --git a/Exec/RegTests/HITDecay/pelelmex_prob_parm.H b/Exec/RegTests/HITDecay/pelelmex_prob_parm.H
@@ -4,6 +4,8 @@
 #include <AMReX_REAL.H>
 #include <AMReX_GpuMemory.H>
 
+#define PELE_PROB_PARM_NEEDS_FREEING
+
 using namespace amrex::literals;
 
 struct ProbParm

diff --git a/Source/PeleLMeX.H b/Source/PeleLMeX.H
@@ -118,6 +118,7 @@ public:
 
   // ReadProblemParameters
   void readProbParm();
+  void freeProbParm();
 
   // ReadGridFile
   void

diff --git a/Source/PeleLMeX.cpp b/Source/PeleLMeX.cpp
@@ -28,7 +28,9 @@ PeleLM::~PeleLM()
 
   closeTempFile();
   typical_values.clear();
-
+#ifdef PELE_PROB_PARM_NEEDS_FREEING
+  freeProbParm();
+#endif
   delete prob_parm;
   The_Arena()->free(prob_parm_d);
   m_initial_ba.clear();