diff options
author | Haniel Barbosa <hanielbbarbosa@gmail.com> | 2021-04-26 11:46:56 -0300 |
---|---|---|
committer | GitHub <noreply@github.com> | 2021-04-26 14:46:56 +0000 |
commit | bff297565a91e020c845d2984a3f0f921f887bba (patch) | |
tree | df2765a0dcbdef1515be758ab8290e91b9c70888 | |
parent | b6d3dc39fcd1cd71d263a4ff6949d77d671c73b9 (diff) |
Fix assertions in SAT solver (#6443)
Due to our recent changes in the unsat core infrastructure we were doing a couple assertions wrong during conflict analysis. This commit fixes them.
-rw-r--r-- | src/prop/minisat/core/Solver.cc | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/src/prop/minisat/core/Solver.cc b/src/prop/minisat/core/Solver.cc index 19f1bff97..971fb95d2 100644 --- a/src/prop/minisat/core/Solver.cc +++ b/src/prop/minisat/core/Solver.cc @@ -2249,8 +2249,8 @@ CRef Solver::updateLemmas() { // Last index in the trail int backtrack_index = trail.size(); - Assert(!options::unsatCores() || needProof() - || lemmas.size() == (int)lemmas_cnf_assertion.size()); + Assert(options::unsatCoresMode() != options::UnsatCoresMode::OLD_PROOF + || lemmas.size() == static_cast<int>(lemmas_cnf_assertion.size())); // Attach all the clauses and enqueue all the propagations for (int j = 0; j < lemmas.size(); ++j) @@ -2334,8 +2334,8 @@ CRef Solver::updateLemmas() { } } - Assert(!options::unsatCores() || needProof() - || lemmas.size() == (int)lemmas_cnf_assertion.size()); + Assert(options::unsatCoresMode() != options::UnsatCoresMode::OLD_PROOF + || lemmas.size() == static_cast<int>(lemmas_cnf_assertion.size())); // Clear the lemmas lemmas.clear(); lemmas_cnf_assertion.clear(); |