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

Fixing yet a source of dependency on alphabetic order in unification. #7257

Merged

Commits on Sep 14, 2018

  1. Fixing yet a source of dependency on alphabetic order in unification.

    This refines even further c24bcae (PR coq#924) and 6304c84:
    - c24bcae fixed the order in the heuristic
    - 6304c84 improved the order by preferring projections
    
    There remained a dependency in the alphabetic order in selecting
    unification candidates. The current commit fixes it.
    
    We radically change the representation of the substitution to invert
    by using a map indexed on the rank in the signature rather than on the
    name of the variable.
    
    More could be done to use numbers further, e.g. for representing
    aliases.
    
    Note that this has consequences on the test-suite (in
    output/Notations.v) as some problems now infer a dependent return
    clause.
    herbelin committed Sep 14, 2018
    Configuration menu
    Copy the full SHA
    99ffdbb View commit details
    Browse the repository at this point in the history

Commits on Sep 15, 2018

  1. Overlay for cross-crypto.

    herbelin committed Sep 15, 2018
    Configuration menu
    Copy the full SHA
    c9c18ed View commit details
    Browse the repository at this point in the history