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

Rewrite the AES parameterized example to show off the new module system #1375

Open
Tracked by #1363
m-yac opened this issue Jun 24, 2022 · 1 comment
Open
Tracked by #1363
Labels
parameterized modules Related to Cryptol's parameterized modules

Comments

@m-yac
Copy link
Contributor

m-yac commented Jun 24, 2022

Currently the examples/param_modules/AES and cryptol-specs AES examples heavily rely on backticked module imports, as they were likely designed to show off this feature. With this feature removed, we should rewrite these examples to show off the new module system, instead of directly adapting them. This would get rid of the clunky code which results from directly adapting these examples, and would add a good larger example of using interface modules to the repo.

@m-yac m-yac added the parameterized modules Related to Cryptol's parameterized modules label Jun 24, 2022
@m-yac
Copy link
Contributor Author

m-yac commented Jul 14, 2022

Once #1373 is incorporated, we should just use that feature instead of rewriting the whole example.

@m-yac m-yac mentioned this issue Jul 14, 2022
8 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
parameterized modules Related to Cryptol's parameterized modules
Projects
None yet
Development

No branches or pull requests

1 participant