-
Notifications
You must be signed in to change notification settings - Fork 26
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
Port to Coq 8.7 and template-coq v8.7 branch #3
Conversation
removed an admit.
of Coq and SquiggleEq.
L4 to L4_2 is de-bruijn to named conversion. Perhaps eval does not syntactically commute with the translation; at least I couldn't prove it. It certainly does that upto alpha equality. Thus a proof that alpha equal values have same overvations was required. coq-paramcoq is now needed. `opam install coq-paramcoq` should suffice. See README.md (8.7 branch) for details.
I made quite a few changes so that CertiCoq looks more like a regular Coq package that can be installed:
This all compiles with @aa755 Could you test on your setup before we switch officially? |
Sure. Will start the build right away. |
|
Ok, I probably should have [mkdir plugin/extraction] somewhere. You do see the extracted .ml's in theories/Extraction after make right? |
Should be fixed now. You can try |
| flcons_wf_e: forall i f e es, exp_wf i e -> isLambda e -> efnlst_wf i es -> efnlst_wf i (eflcons f e es)
Yes, it is fixed now. |
Cool!
…On Sat, Feb 17, 2018 at 2:09 PM Abhishek Anand ***@***.***> wrote:
Yes, it is fixed now. make install worked and so did Certicoq Compile foo.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#3 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAGARXYDEyDXBoN0ztG80Sr_IkfUIjc_ks5tVs-EgaJpZM4RyFe2>
.
|
We are ready to move to 8.7, aren't we? Should we make 8.7 the default branch? or merge this PR? |
Randy, please check that the following file, which had merge conflicts, is okay theories/L2k_noCnstrParams/compile.v
Cool, thanks for the merge @aa755 ! |
This is the port to Coq 8.7 and the current squiggle-eq and template-coq 8.7 branches. I needed admits in L4_5_to_L5.v due to changes in squiggle-eq apparently. Thanks to the new coq_makefile features I was able to remove an Obj.magic in CertiCoq's plugin, we really compile up to the interface of Template-Coq's produced OCaml Ast module now. The rest is:
Require Import FunInd.
in files that refer tofunctional induction
.That was just a few hours of work :) The plugin works as expected.