-
Notifications
You must be signed in to change notification settings - Fork 22
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
Fix dependencies bounded integers #808
base: main
Are you sure you want to change the base?
Conversation
90c8b0e
to
6e3485d
Compare
This PR has been marked as stale due to a lack of activity for 60 days. If you believe this pull request is still relevant, please provide an update or comment to keep it open. Otherwise, it will be closed in 7 days. |
This PR has been closed due to a lack of activity since being marked as stale. If you believe this pull request is still relevant, please reopen it with an update or comment. |
I think we should have merged this one. |
6e3485d
to
c9e7055
Compare
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.
Do we actually use bounded integers now? I thought it didn't hit the mark. If we do, let's get it in. If not, let's talk about what to do with this.
Let's meet to discuss bounded integers in general. We may want to revert some of this. |
@W95Psp lets get it fixed and merged |
c9e7055
to
985057a
Compare
985057a
to
e086d99
Compare
I fixed the PR, we should be able to merge it now |
This PR:
hax_bounded_integers::refinement_int
(macro_rules rely on dependencies which were then not available at callsite)