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

fix: use more precise types for move_requires_* #3449

Merged
merged 2 commits into from
Sep 7, 2024
Merged

fix: use more precise types for move_requires_* #3449

merged 2 commits into from
Sep 7, 2024

Conversation

TWal
Copy link
Contributor

@TWal TWal commented Sep 6, 2024

... and add move_requires_4, a companion to forall_intro_4 that was missing.

I noticed that move_requires_n don't use dependent types like forall_intro_n do, which I needed, hence this PR.

@mtzguido
Copy link
Member

mtzguido commented Sep 7, 2024

Thanks Théophile! This looks good. Just that somehow it makes LowStar.Monotonic.Buffer go into an infinite loop... I'll take a look but I killed the CI for now.

@mtzguido
Copy link
Member

mtzguido commented Sep 7, 2024

Apparently randomly, this query was causing Z3 to go into an infinite loop without burning rlimit. Some tweaks avoid. It's a good reminder to bump the default Z3 version.

Thanks again, sorry for the snafu.

@mtzguido mtzguido enabled auto-merge September 7, 2024 04:44
@mtzguido mtzguido merged commit 6c8c813 into FStarLang:master Sep 7, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants