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

[TC] HO unification #571

Closed
wants to merge 81 commits into from
Closed

[TC] HO unification #571

wants to merge 81 commits into from

Conversation

FissoreD
Copy link
Collaborator

@FissoreD FissoreD commented Jan 8, 2024

  • univ-instance is replaced with an elpi fresh var to ease unification
  • allow multi trace with Elpi Trace Browser
  • HO unification of elpi if a term is in pattern fragment
  • compile instance methods which are coercions (an example is here)

the univ-instance of a pglobal term is replaced with a fresh elpi
variable to help unification. This strategy is also used to compile
instances whose type is [pglobal ClassGR <<...>>].
@FissoreD FissoreD requested a review from gares January 8, 2024 17:12
gares and others added 13 commits January 9, 2024 10:36
A class with a parameter P of functional type,
could possibly have an eta-expansion which could make
the search for instance to fail.
To solve solve this problem, a new clause is generated
at the end of the db where a eta-contraction is tried on
P and if the new term P' is different from P, a new search
is run using P'
add-unfold GR :-
coq.error "[TC]" GR "is not a constant".
main L :-
ErrMsg = "[TC] TC.Unfold accepts a list of string is accepted",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
ErrMsg = "[TC] TC.Unfold accepts a list of string is accepted",
ErrMsg = "[TC] TC.Unfold takes a list of strings",

@FissoreD FissoreD changed the title [TC] generalize univ-instance [TC] HO unification Jan 22, 2024
@FissoreD FissoreD closed this Apr 3, 2024
@FissoreD FissoreD deleted the pglobal1 branch April 3, 2024 11:13
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