-
Notifications
You must be signed in to change notification settings - Fork 20
chore: refactor Comb
parser and remove unnecessary types
#1237
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
Conversation
Alive Statistics: 90 / 93 (3 failed) |
bitwuzla proved and leanSAT failed theorem 3 in file gexact_proof.lean |
Alive Statistics: 90 / 93 (3 failed) |
Alive Statistics: 90 / 93 (3 failed) |
bitwuzla proved and leanSAT failed theorem 3 in file gexact_proof.lean |
Alive Statistics: 90 / 93 (3 failed) |
bitwuzla proved and leanSAT failed theorem 3 in file gexact_proof.lean |
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.
despite setting ϕ to 0, I can't seem to remove the check on the mvar in MkTy, what am I missing?
Not sure, the code look like it'd ought to work. What error or you seeing?
I hacked a bit the arguments management for n-ary ops - I believe there are smoother ways to do that, could you please suggest how to do it better?
I left a suggestion
I would like to throw a more informative error for Comb.mux when the third arg is not .bitvec 1 (condition), since as of now a very generic type error is raised. My solution would be to nest a further match in the parser, which I believe is what we're trying to avoid with the new infrastructure. What's the best way to proceed?
Adding an extra match would be fine, I think. I left a suggestion with a rough outline of what I'd write!
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.
LGTM, modulo a error message phrasing
Alive Statistics: 90 / 93 (3 failed) |
This PR refactors the
Comb
parser according to #1208, removes unnecessary types (so that everything is a bitvector operation) and removes support forconcat
, since it can't be supported withoutHVector
/hList
/similar.