diff options
author | Clark Barrett <barrett@cs.nyu.edu> | 2012-06-10 14:43:58 +0000 |
---|---|---|
committer | Clark Barrett <barrett@cs.nyu.edu> | 2012-06-10 14:43:58 +0000 |
commit | 48f837c9d53e2e93b14786943f9961228e0d9933 (patch) | |
tree | 6caf8bc4dc9e6d937b7db30587257504be632589 /src/theory/bv | |
parent | 103d6a6aad30410f8d7546c25c1f5e67f1c334d7 (diff) |
Added a very fruitful assertion to the rewriter: checks that rewriting after "REWRITE_DONE" is idempotent
Found several problems where this is not the case and fixed them
Diffstat (limited to 'src/theory/bv')
-rw-r--r-- | src/theory/bv/theory_bv_rewrite_rules_normalization.h | 8 | ||||
-rw-r--r-- | src/theory/bv/theory_bv_rewriter.cpp | 2 |
2 files changed, 8 insertions, 2 deletions
diff --git a/src/theory/bv/theory_bv_rewrite_rules_normalization.h b/src/theory/bv/theory_bv_rewrite_rules_normalization.h index 197134b6a..4fa96c231 100644 --- a/src/theory/bv/theory_bv_rewrite_rules_normalization.h +++ b/src/theory/bv/theory_bv_rewrite_rules_normalization.h @@ -429,8 +429,11 @@ Node RewriteRule<SolveEq>::apply(TNode node) { // If both constants are nonzero, combine on right, otherwise leave them where they are if (rightConst != zero) { + leftConst = zero; rightConst = rightConst - leftConst; - childrenRight.push_back(utils::mkConst(rightConst)); + if (rightConst != zero) { + childrenRight.push_back(utils::mkConst(rightConst)); + } } else if (leftConst != zero) { childrenLeft.push_back(utils::mkConst(leftConst)); @@ -540,6 +543,9 @@ Node RewriteRule<SolveEq>::apply(TNode node) { newRight = utils::mkSortedNode(kind::BITVECTOR_PLUS, childrenRight); } + Assert(newLeft == Rewriter::rewrite(newLeft)); + Assert(newRight == Rewriter::rewrite(newRight)); + if (newLeft == newRight) { Assert (newLeft == utils::mkConst(size, (unsigned)0)); return utils::mkTrue(); diff --git a/src/theory/bv/theory_bv_rewriter.cpp b/src/theory/bv/theory_bv_rewriter.cpp index d6de6edbd..47725444d 100644 --- a/src/theory/bv/theory_bv_rewriter.cpp +++ b/src/theory/bv/theory_bv_rewriter.cpp @@ -107,7 +107,7 @@ RewriteResponse TheoryBVRewriter::RewriteUle(TNode node, bool preregister){ RewriteRule<UleZero>, RewriteRule<UleSelf> >::apply(node); - return RewriteResponse(REWRITE_DONE, resultNode); + return RewriteResponse(resultNode == node ? REWRITE_DONE : REWRITE_AGAIN, resultNode); } RewriteResponse TheoryBVRewriter::RewriteSle(TNode node, bool preregister){ |