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

Fixes for https://github.com/FStarLang/FStar/pull/3072 #103

Merged
merged 3 commits into from
Oct 30, 2023

Conversation

mtzguido
Copy link
Member

These changes are needed to keep the build working after the F* PR is merged.

These two definitions failed after the fix in
FStarLang/FStar#3072. The assertions make it
work
I've seen this proof randomly regress several times, with the error:

* Error 228 at Steel.ST.Array.Util.fst(68,4-68,25):
  - Tactic failed
  - "apply_lemma: Cannot instantiate lemma FStar.Tactics.V2.Logic.fa_intro_lem (with postcondition: Prims.squash (forall (x: Steel.Effect.Common.vprop).
      Steel.Effect.Common.equiv (Steel.Effect.Common.VStar (*?u4892*) _ (*?u4888*) _) (*?u4880*) _ ==>
      (*?u4872*) _)) to match goal (Prims.squash (forall (x2: Steel.Effect.Common.vprop).
      Steel.Effect.Common.equiv (Steel.Effect.Common.VStar x x2) x ==>
      Steel.Effect.Common.equiv (Steel.Effect.Common.VStar x2 x)
        (Steel.Effect.Common.VStar (Steel.Effect.Common.VStar Steel.Effect.Common.emp x)
            Steel.Effect.Common.emp)))"

The latest one being when trying to merge the normalizer fix in
FStarLang/FStar#3072. Apparently, just adding an
extra unit makes it work, so I am just doing that to unblock the F* PR.
@mtzguido mtzguido enabled auto-merge October 30, 2023 18:03
@mtzguido mtzguido merged commit 06aa03c into FStarLang:main Oct 30, 2023
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.

1 participant