One bug in our port and another bug in the original version
authorbdemsky <bdemsky@uci.edu>
Tue, 5 Sep 2017 07:56:27 +0000 (00:56 -0700)
committerbdemsky <bdemsky@uci.edu>
Tue, 5 Sep 2017 07:56:27 +0000 (00:56 -0700)
src/Backend/cnfexpr.cc
src/Backend/constraint.cc

index 1aa6cf0..fb01d3f 100644 (file)
@@ -190,7 +190,7 @@ void copyCNF(CNFExpr *This, CNFExpr *expr, bool destroy) {
 
 void conjoinCNFExpr(CNFExpr *This, CNFExpr *expr, bool destroy) {
        if (expr->litSize == 0) {
-               if (!This->isTrue) {
+               if (!expr->isTrue) {
                        clearCNFExpr(This, false);
                }
                if (destroy) {
index 0372536..61fba12 100644 (file)
@@ -839,8 +839,8 @@ CNFExpr *produceDisjunction(CNF *cnf, Edge e) {
                                int eL = argExp->litSize;                       // lits in argument
                                int aC = getClauseSizeCNF(accum);               // clauses in accum
                                int eC = getClauseSizeCNF(argExp);      // clauses in argument
-
-                               if (eC > CLAUSE_MAX || (eL * aC + aL * eC > eL + aC + aL + aC)) {
+                               //POSSIBLE BUG #2 (eL * aC +aL * eC > eL +aC +aL +aC)
+                               if (eC > CLAUSE_MAX || (eL * aC + aL * eC > eL + eC + aL + aC)) {
                                        disjoinCNFLit(accum, introProxy(cnf, arg, argExp, isNegEdge(arg)));
                                } else {
                                        disjoinCNFExpr(accum, argExp, destroy);