diff options
author | lianah <lianahady@gmail.com> | 2013-01-10 20:44:58 -0500 |
---|---|---|
committer | lianah <lianahady@gmail.com> | 2013-01-10 20:44:58 -0500 |
commit | f8c88fa4b7b9b2d59f48d0e33f1344196a06f5da (patch) | |
tree | 97ea6b2a2304d867cceebec6c9494ec1fcbd749e /src/theory/bv/theory_bv.cpp | |
parent | 590e7f438dacbee1c349f431316e918de43e5a8e (diff) |
fixed most bugs and added paranoid assertions
Diffstat (limited to 'src/theory/bv/theory_bv.cpp')
-rw-r--r-- | src/theory/bv/theory_bv.cpp | 31 |
1 files changed, 16 insertions, 15 deletions
diff --git a/src/theory/bv/theory_bv.cpp b/src/theory/bv/theory_bv.cpp index bb16e00ce..5af5b2d23 100644 --- a/src/theory/bv/theory_bv.cpp +++ b/src/theory/bv/theory_bv.cpp @@ -127,23 +127,24 @@ void TheoryBV::check(Effort e) // FIXME: this is not quite correct as it does not take into account cardinality constraints - if (d_coreSolver.isCoreTheory()) { + //if (d_coreSolver.isCoreTheory()) { // paranoid check to make sure results of bit-blaster agree with slicer for core theory - if (inConflict()) { - d_conflict = false; - d_bitblastSolver.addAssertions(new_assertions, Theory::EFFORT_FULL); - Assert (inConflict()); - } else { - d_bitblastSolver.addAssertions(new_assertions, Theory::EFFORT_FULL); - Assert (!inConflict()); - } + // if (inConflict()) { + // d_conflict = false; + // d_bitblastSolver.addAssertions(new_assertions, Theory::EFFORT_FULL); + // Assert (inConflict()); + // } else { + // d_bitblastSolver.addAssertions(new_assertions, Theory::EFFORT_FULL); + // Assert (!inConflict()); + // } + //} + //else { + + if (!inConflict() && !d_coreSolver.isCoreTheory()) { + // sending assertions to the bitblast solver if it's not just core theory + d_bitblastSolver.addAssertions(new_assertions, e); } - else { - if (!inConflict()) { - // sending assertions to the bitblast solver - d_bitblastSolver.addAssertions(new_assertions, e); - } - } + if (inConflict()) { sendConflict(); } |