FStarLang / FStar

A Proof-oriented Programming Language
https://fstar-lang.org
Apache License 2.0
2.7k stars 234 forks source link

fix: use more precise types for `move_requires_*` #3449

Closed TWal closed 2 months ago

TWal commented 2 months ago

... 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 commented 2 months ago

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 commented 2 months ago

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.