-
Notifications
You must be signed in to change notification settings - Fork 3
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
Adding DataPlaneSpec to the IO-spec #230
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Mostly minor comments. I guess we can merge it after you are done with them
Co-authored-by: João Pereira <joaopereira.19@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
We can merge it when the tests go through, unless @Dspil has a different opinion
The IO-spec is rewritten such that it depends on an abstract verison of the dataplane. This is needed to provide additional ghost state. Also the IO-spec uses now the match clause, making it easier to read and a closer match to the IO-spec from Isabelle. Finally, all unrequired functions are commented out.