Skip to content

Commit

Permalink
missing initialization of m_user_propagator, disable unsound in-proce…
Browse files Browse the repository at this point in the history
…ssing in pb_solver

Signed-off-by: Nikolaj Bjorner <nbjorner@microsoft.com>
  • Loading branch information
NikolajBjorner committed Feb 23, 2022
1 parent dc110f1 commit 7b4f1ed
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 2 deletions.
9 changes: 7 additions & 2 deletions src/sat/smt/pb_solver.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -2637,6 +2637,10 @@ namespace pb {
* add ~root(~l) to c, k <- k + 1
*/
void solver::unit_strengthen() {
return;

// TODO - this is unsound exposed by 4_21_21_-1.txt

sat::big big(s().m_rand);
big.init(s(), true);
for (unsigned sz = m_constraints.size(), i = 0; i < sz; ++i)
Expand All @@ -2646,7 +2650,8 @@ namespace pb {
}

void solver::unit_strengthen(sat::big& big, constraint& p) {
if (p.lit() != sat::null_literal) return;
if (p.lit() != sat::null_literal)
return;
unsigned sz = p.size();
for (unsigned i = 0; i < sz; ++i) {
literal u = p.get_lit(i);
Expand Down Expand Up @@ -2694,8 +2699,8 @@ namespace pb {
}
}
++m_stats.m_num_big_strengthenings;
constraint* c = add_pb_ge(sat::null_literal, wlits, b, p.learned());
p.set_removed();
add_pb_ge(sat::null_literal, wlits, b, p.learned());
return;
}
}
Expand Down
1 change: 1 addition & 0 deletions src/smt/smt_context.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -211,6 +211,7 @@ namespace smt {
new_ctx->m_is_auxiliary = true;
new_ctx->set_logic(l == nullptr ? m_setup.get_logic() : *l);
copy_plugins(*this, *new_ctx);
new_ctx->copy_user_propagator(*this);
return new_ctx;
}

Expand Down

0 comments on commit 7b4f1ed

Please sign in to comment.