Skip to content

Adapt to rocq#19987 #1532

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Adapt to rocq#19987 #1532

wants to merge 1 commit into from

Conversation

Tragicus
Copy link
Collaborator

Motivation for this change

rocq-prover/rocq#19987 refolds terms before using them to instantiate evars. In classical/functions.v, there is a lemma (image_subP) that uses homomorphism_1, which introduces eta-expansions. Due to unfoldings in the previous algorithm, these eta-expansions were removed, whereas they are not now, which makes rewrite's matching algorithm fail. In theories/function_spaces, there is a second order unification problem which is solved differently.

Checklist
  • added corresponding entries in CHANGELOG_UNRELEASED.md
  • added corresponding documentation in the headers

Reference: How to document

Merge policy

As a rule of thumb:

  • PRs with several commits that make sense individually and that
    all compile are preferentially merged into master.
  • PRs with disorganized commits are very likely to be squash-rebased.
Reminder to reviewers

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