-
Notifications
You must be signed in to change notification settings - Fork 20
Conversation
…ns use the correct order.
…ure all three of those functions are defined.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I left a few more comments, but I think this PR is almost ready to be merged.
I think you should run the SyGuS 2.0 files that I sent and make sure we still pass those benchmarks.
I ran it against the SyGuS 2.0 files and it passes all of them except for 2 which don't finish running. For some reason the bash script says "unknown flag -t" and doesn't let me set a time limit. |
Great, thanks! I am not sure about the "-t" issue -- I'll try it locally and let you know. This PR is ready to merge, but I will merge it after #11 since that's a much larger PR and would need some changes to benchmarks if this is merged first. |
Does this pull request supports whole SyGuS language 2.0?
The contents of
|
Hello @Kakadu, No, LoopInvGen currently does not support user-defined data types. It fully supports Boolean and integer data types, provides minimal support for arrays (we are working on extending it) and we are currently adding bitvector support (PR #13). There is an old branch that adds support for string type, but it's stalled because of parsing issues with Jane streets sexpression library. |
declare_primed_var
, anddeclare_fun
.let
terms are written in the SMT-LIB compliant way.