diff options
| author | Andres Noetzli <noetzli@stanford.edu> | 2017-05-15 09:50:45 -0700 |
|---|---|---|
| committer | Andres Noetzli <noetzli@stanford.edu> | 2017-05-15 09:50:45 -0700 |
| commit | fdef54aaa36793cb9314dea3830879e051db2aa8 (patch) | |
| tree | e891a13b92dd42ad578b90340e61637f3651ace4 | |
| parent | 84af3731aa40e2e6f9281827af87350a2cb44ea1 (diff) | |
Fix minor bug in sets rewriter
As reported by Coverity, one of the switches in the sets rewriter had a missing
break. This could lead to an assertion failure when rewriting the cardinality
of a transpose as in the test case included in this commit.
| -rw-r--r-- | src/theory/sets/theory_sets_rewriter.cpp | 1 | ||||
| -rw-r--r-- | test/regress/regress0/rels/Makefile.am | 3 | ||||
| -rw-r--r-- | test/regress/regress0/rels/card_transpose.cvc | 6 |
3 files changed, 9 insertions, 1 deletions
diff --git a/src/theory/sets/theory_sets_rewriter.cpp b/src/theory/sets/theory_sets_rewriter.cpp index 3590fc62d..e42a3347d 100644 --- a/src/theory/sets/theory_sets_rewriter.cpp +++ b/src/theory/sets/theory_sets_rewriter.cpp @@ -328,6 +328,7 @@ RewriteResponse TheorySetsRewriter::postRewrite(TNode node) { NodeManager::currentNM()->mkNode( kind::CARD, NodeManager::currentNM()->mkNode( kind::INTERSECTION, node[0][0], node[0][1] ) ) ); return RewriteResponse(REWRITE_DONE, ret ); } + break; } case kind::TRANSPOSE: { if(node[0].getKind() == kind::TRANSPOSE) { diff --git a/test/regress/regress0/rels/Makefile.am b/test/regress/regress0/rels/Makefile.am index d1c035371..7f772a8e1 100644 --- a/test/regress/regress0/rels/Makefile.am +++ b/test/regress/regress0/rels/Makefile.am @@ -112,7 +112,8 @@ TESTS = \ joinImg_1_1.cvc \ joinImg_1.cvc \ joinImg_2_1.cvc \ - joinImg_2.cvc + joinImg_2.cvc \ + card_transpose.cvc # unsolved : garbage_collect.cvc diff --git a/test/regress/regress0/rels/card_transpose.cvc b/test/regress/regress0/rels/card_transpose.cvc new file mode 100644 index 000000000..bde7fe53e --- /dev/null +++ b/test/regress/regress0/rels/card_transpose.cvc @@ -0,0 +1,6 @@ +% EXPECT: unknown (INCOMPLETE) +OPTION "logic" "ALL_SUPPORTED"; +IntPair: TYPE = [INT, INT]; +x : SET OF IntPair; +ASSERT (CARD(TRANSPOSE(x)) > 0); +CHECKSAT; |