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

Poulet4 interpreter fixes #1

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

hackedy
Copy link
Collaborator

@hackedy hackedy commented Sep 28, 2022

Hi, I have some changes in the branch poulet4-interpreter over in the Petr4 repo that I would like to merge into poulet4. I took a look to see how much my changes break Verifiable P4 and in this PR I fix the issues, so far as I can tell.

Can someone take a look at what I've changed and let me know if it builds on your machine (after installing poulet4 from the poulet4-interpreter branch)? I am seeing some build problems that don't appear to be due to the poulet4-interpreter changes, e.g.,

File "./examples/sbf/FlowProperty.v", line 42, characters 63-65:
Error:
In environment
f : list packet
i : Z
The term "Tc" has type "Z -> Z -> Z" while it is expected to have type "Z".

And this, which I assume means I'm missing some VST-related opam package?

File "./examples/sbf/flow_proof.v", line 1, characters 0-35:
Error: Cannot find a physical path bound to logical path VST.floyd.proofauto.

@hackedy
Copy link
Collaborator Author

hackedy commented Sep 29, 2022

Build hangs in ./examples/bloomfilter/verif.v, so my changes might have accidentally made the genv terms really big.

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.

1 participant