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

CN: Simplify arguments to provable(withUnknown) #902

Open
dc-mak opened this issue Mar 4, 2025 · 0 comments
Open

CN: Simplify arguments to provable(withUnknown) #902

dc-mak opened this issue Mar 4, 2025 · 0 comments
Labels
cn solver Related to the SMT solver backend technical debt Something for internal cleanup

Comments

@dc-mak
Copy link
Contributor

dc-mak commented Mar 4, 2025

          I'm also a bit confused about all of the arguments to `provable`. I have a `Global.t` and a list of constraints that I want to check the consistency of. Christopher previously recommended adding all of the constraints to the solver as assumptions; should I still do that, or just `and` them together and make them the last argument? Would I leave the assumptions argument empty then? For `simp_ctxt`, can I follow the logic of `make_simp_ctxt` in typing.ml?

Originally posted by @cassiatorczon in #829 (comment)

@dc-mak dc-mak added cn solver Related to the SMT solver backend technical debt Something for internal cleanup labels Mar 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cn solver Related to the SMT solver backend technical debt Something for internal cleanup
Projects
None yet
Development

No branches or pull requests

1 participant