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

Proof failures with (unsupported) Coq 8.12 #8

Closed
blaxill opened this issue Apr 15, 2021 · 3 comments
Closed

Proof failures with (unsupported) Coq 8.12 #8

blaxill opened this issue Apr 15, 2021 · 3 comments

Comments

@blaxill
Copy link

blaxill commented Apr 15, 2021

Some changes in Coq 8.12 tactics break proofs in a few places.

  353         * f_equal.
  354           rewrite N.mul_mod_distr_l by (destruct sz; cbn; lia).  
  355           reflexivity.

This can be fixed by an intermediate remember:

  353         * f_equal.               
  354           rewrite N.mul_mod_distr_l ; (destruct sz; cbn; try lia).  
  355           remember (2^(Pos.of_succ_nat sz))%positive; lia

This seems odd to me (I can't find a relevant changelog for lia), does the Koika team think this is a Coq issue?

@cpitclaudel
Copy link
Collaborator

Thanks! I tried to make a backwards-compatible fix. Can you give it a try?

@blaxill
Copy link
Author

blaxill commented Apr 16, 2021

Thanks, I can confirm its building for me now with 8.12 and 8.13. (There are still the errors from #7 from the multiplercorrectness example).

@cpitclaudel
Copy link
Collaborator

Thanks a lot. #7 is on my list

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

No branches or pull requests

2 participants