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

False positive on QF_BVFPLRA #74

Closed
aytey opened this issue Jul 19, 2021 · 4 comments · Fixed by #79
Closed

False positive on QF_BVFPLRA #74

aytey opened this issue Jul 19, 2021 · 4 comments · Fixed by #79
Labels

Comments

@aytey
Copy link

aytey commented Jul 19, 2021

For this file:

(set-logic QF_BVFPLRA)
; (set-logic QF_LRA)
(declare-fun honk () Real)
(assert (= honk 0.75))
(check-sat)
(exit)

Dolmen reports:

File "./lra.smt2", line 4, character 16-20:
Error Real literals are not part of the Floats specification.

However, the logic LRA has been enabled (and Dolmen does not report "unknown logic"). If you comment the first line and uncomment the second (i.e., so LRA is the enabled logic), Dolmen happily accepts this file.

@Gbury
Copy link
Owner

Gbury commented Jul 19, 2021

That's a bug indeed, will look at it as soon as possible.

@Gbury Gbury added the bug label Jul 19, 2021
@Gbury
Copy link
Owner

Gbury commented Jul 20, 2021

related to #43

@Gbury
Copy link
Owner

Gbury commented Jul 20, 2021

This will be solved simply by stopping to accept real literals in the float theory. However, this problem could also be caused by bitvector literals, so I have a question: are bitvectors extended literals (i.e. (_ bv13 32)) officially allowed in the float theory ?

@aytey
Copy link
Author

aytey commented Jul 20, 2021

This will be solved simply by stopping to accept real literals in the float theory. However, this problem could also be caused by bitvector literals, so I have a question: are bitvectors extended literals (i.e. (_ bv13 32)) officially allowed in the float theory ?

I'm faaaaarrrrrrr from an expert, but I think not: only #bX and #xX are supported in the FP theory.

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