Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

rewriter.push_ite_arith rewriter.eq2ineq fp.xform.elim_term_ite rewriter.arith_ineq_lhs with horn Assertion violation at ../src/muz/transforms/dl_mk_elim_term_ite.cpp Line: 168 #3849

Closed
muchang opened this issue Apr 7, 2020 · 3 comments
Assignees

Comments

@muchang
Copy link

muchang commented Apr 7, 2020

Hi,
For this formula:

(set-option :rewriter.push_ite_arith true)
(set-option :rewriter.eq2ineq true)
(set-option :fp.xform.elim_term_ite true)
(set-option :rewriter.arith_ineq_lhs true)
(assert
 (forall ((a Bool) (b Bool) (c Int) (d Int))
 (= d (ite (= a b) 0 (ite (= c 0) (+ c 1) c)))))
(check-sat-using horn)

Z3 throws out an assertion violation:

ASSERTION VIOLATION
File: ../src/muz/transforms/dl_mk_elim_term_ite.cpp
Line: 168
!has_term_ite(fml2)
(C)ontinue, (A)bort, (S)top, (T)hrow exception, Invoke (G)DB

OS: Ubuntu 18.04
Commit: cb13641

@agurfinkel
Copy link
Collaborator

comment to self: check why the assertion was failing

@NikolajBjorner
Copy link
Contributor

comment to self: check why the assertion was failing

The assertion is failing in part because it is an ite where
the then and else branch have type Bool.
The blast_term_ite_tactic does not expand such subterms because it will not help the SMT solver to change these terms. The SMT solver clausifies ite formulas already and there is no point in hoisting them.
Perhaps the transformation rule should also allow such Boolean ite expressions as the potential advantages of removing ite expressions possibly for arithmetic reasoning.

NikolajBjorner added a commit that referenced this issue Apr 10, 2020
Signed-off-by: Nikolaj Bjorner <nbjorner@microsoft.com>
@agurfinkel
Copy link
Collaborator

moved assert into a trace print in d53e30e

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants