Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
fix unsoundness issue due to book-keeping changes for whether the solver uses assumptions.
  • Loading branch information
NikolajBjorner committed Aug 18, 2022
1 parent 1a5503c commit 8e167aa
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/sat/sat_solver.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -2416,7 +2416,7 @@ namespace sat {
m_conflict_lvl = get_max_lvl(m_not_l, m_conflict, unique_max);
justification js = m_conflict;

if (m_conflict_lvl <= 1 && tracking_assumptions()) {
if (m_conflict_lvl <= 1 && (!m_assumptions.empty() || !m_user_scope_literals.empty())) {
TRACE("sat", tout << "unsat core\n";);
resolve_conflict_for_unsat_core();
return l_false;
Expand Down

0 comments on commit 8e167aa

Please sign in to comment.