Skip to content

Commit

Permalink
comment
Browse files Browse the repository at this point in the history
  • Loading branch information
Giles authored and Martin Suda committed Oct 6, 2014
1 parent ddf8104 commit 2906ef0
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 2 deletions.
2 changes: 1 addition & 1 deletion SAT/MinimizingSolver.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,7 @@ void MinimizingSolver::addClauses(SATClauseIterator cit, bool onlyPropagate)
newClauses.loadFromIterator(cit);

//we need to filter out the empty clause -- it won't have any influence on our algorithm
//(as it will make the problem unsat and we process only satisfiale assignment), but it'd
//(as it will make the problem unsat and we process only satisfiale assignment), but it
//is a corner case that needs to be handled
_unprocessed.loadFromIterator(
getFilteredIterator(SATClauseStack::BottomFirstIterator(newClauses), isNonEmptyClause));
Expand Down
4 changes: 3 additions & 1 deletion Saturation/SSplitter.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -457,7 +457,7 @@ bool SSplitter::shouldAddClauseForNonSplittable(Clause* cl, unsigned& compName,
}

if(_congruenceClosure && cl->length()==1 && (*cl)[0]->ground() && cl->splits()->isEmpty()) {
//we add ground clauses if we use congruence closure...
//we add ground unit clauses if we use congruence closure...
compName = getComponentName(cl->length(), cl->literals(), cl, compCl);
RSTAT_CTR_INC("ssat_ground_clauses_for_congruence");
return true;
Expand Down Expand Up @@ -490,6 +490,8 @@ bool SSplitter::shouldAddClauseForNonSplittable(Clause* cl, unsigned& compName,
}
ASS_NEQ(cl,compCl);

// We only reach here if cl already exists as a component

return true;
}

Expand Down

0 comments on commit 2906ef0

Please sign in to comment.