Skip to content
Snippets Groups Projects

Set the priority of the rewrite relation for equiv to not take precedence over...

Merged Matthieu Sozeau requested to merge mattam82/stdpp:pr-13969-take2 into master
All threads resolved!

Set the priority of the rewrite relation for equiv to not take precedence over the eq instance from Coq. This is more backwards compatible performance-wise with the old resolution where eq was favored (otherwise we get a 70% slowdown in perennial).

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Robbert Krebbers resolved all threads

    resolved all threads

  • added 1 commit

    • 4a849876 - Apply 1 suggestion(s) to 1 file(s)

    Compare with previous version

  • Thanks. LGTM. Merging.

  • Robbert Krebbers enabled an automatic merge when the pipeline for 4a849876 succeeds

    enabled an automatic merge when the pipeline for 4a849876 succeeds

  • Robbert Krebbers canceled the automatic merge

    canceled the automatic merge

  • Robbert Krebbers
  • Robbert Krebbers resolved all threads

    resolved all threads

  • added 1 commit

    • 86140f3c - Apply 1 suggestion(s) to 1 file(s)

    Compare with previous version

  • Robbert Krebbers enabled an automatic merge when the pipeline for 86140f3c succeeds

    enabled an automatic merge when the pipeline for 86140f3c succeeds

  • Author Contributor

    I guess automatic merge failed while the pipeline succeeded.

  • Robbert Krebbers canceled the automatic merge

    canceled the automatic merge

  • Robbert Krebbers resolved all threads

    resolved all threads

  • mentioned in commit 607ee2b1

  • Thanks for catching that @mattam82. I was waiting for the merge email to update Iris (but that email never happened since something went wrong here...).

  • Iris has been updated to use a version of std++ that includes this MR.

  • Please register or sign in to reply
    Loading