cleanup more in dependent_expr_state_tactic to reduce mem consumption

Z3Prover · Dec 9, 2022 · c6f9c09 · c6f9c09
1 parent ca6fed8
commit c6f9c09
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 4 deletions.
diff --git a/src/ast/simplifiers/dependent_expr.h b/src/ast/simplifiers/dependent_expr.h
@@ -32,7 +32,6 @@ class dependent_expr {
         m_fml(fml),
         m_proof(p),
         m_dep(d) {
-        SASSERT(fml);
         m.inc_ref(fml);
         m.inc_ref(d);
         m.inc_ref(p);

diff --git a/src/tactic/dependent_expr_state_tactic.h b/src/tactic/dependent_expr_state_tactic.h
@@ -49,7 +49,7 @@ class dependent_expr_state_tactic : public tactic, public dependent_expr_state {
         m(m),
         m_params(p),
         m_factory(f),
-        m_dep(m, m.mk_true(), nullptr, nullptr)
+        m_dep(m, nullptr, nullptr, nullptr)
     {}
 
     /**
@@ -115,13 +115,13 @@ class dependent_expr_state_tactic : public tactic, public dependent_expr_state {
         }
         catch (rewriter_exception& ex) {
             throw tactic_exception(ex.msg());
-        }       
+        }
         m_goal->elim_true();
         m_goal->elim_redundancies();
         m_goal->inc_depth();
         if (in->models_enabled())
             in->add(m_model_trail->get_model_converter().get());
-        result.push_back(in.get());   
+        result.push_back(in.get());
         cleanup();
     }
 
@@ -130,6 +130,8 @@ class dependent_expr_state_tactic : public tactic, public dependent_expr_state {
             m_simp->collect_statistics(m_st);
         m_simp = nullptr;
         m_model_trail = nullptr;
+        m_goal = nullptr;
+        m_dep = dependent_expr(m, nullptr, nullptr, nullptr);
     }
 
     void collect_statistics(statistics & st) const override {