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

Fatal failure on String formula at prop/cnf_stream.cpp:282 #4379

Closed
rainoftime opened this issue Apr 22, 2020 · 0 comments · Fixed by #4380
Closed

Fatal failure on String formula at prop/cnf_stream.cpp:282 #4379

rainoftime opened this issue Apr 22, 2020 · 0 comments · Fixed by #4380
Assignees
Labels

Comments

@rainoftime
Copy link

Related #4340
That test case is closed because it uses proof, which is not supported.


Hi,
For the following formula:

(declare-const i7 Int)
(declare-const Str8 String)
(declare-const Str17 String)
(assert (distinct (str.++ "" "rvhhcn" "" Str8 (int.to.str 56)) (str.++ "" (int.to.str i7) "" Str17) Str17))
(check-sat)

CVC4 throws out a fatal failure:

Fatal failure within CVC4::prop::SatLiteral CVC4::prop::CnfStream::getLiteral(CVC4::TNode) at /home/peisen/test/tofuzz/CVC4/src/prop/cnf_stream.cpp:282
Check failure

 d_nodeToLiteralMap.contains(node)
Literal not in the CNF Cache: (= (int.to.str i7) (str.++ dc_spt_7 dc_spt_rem_8))

Aborted (core dumped)

OS: Ubuntu 16.04
Commit: 2a38d48

@rainoftime rainoftime changed the title Fatal failure at prop/cnf_stream.cpp:282 Fatal failure on String formula at prop/cnf_stream.cpp:282 Apr 22, 2020
@ajreynol ajreynol added the bug label Apr 22, 2020
@ajreynol ajreynol self-assigned this Apr 22, 2020
4tXJ7f pushed a commit that referenced this issue Apr 22, 2020
Fixes #4379. This was caused by a splitting lemma rewriting to a conjunction, being processed as a fact, and having a pending phase requirement sent out assuming the inference was to be processed as a lemma. This forces 2 of the splits in the core solver to be always processed as lemmas.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants