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

Break cyclic build dependency #323

Merged
merged 15 commits into from
Sep 10, 2024
Merged

Conversation

jan-ferdinand
Copy link
Member

@jan-ferdinand jan-ferdinand commented Sep 4, 2024

  • make many small crates without cycles
    • constraint circuit
    • ISA
    • AIR
    • constraint builder
    • Triton VM
  • add README.md to each crate
  • shrink public interfaces
  • make public exports of all the new crates
  • publish the new crates

@jan-ferdinand jan-ferdinand force-pushed the circuit_builder_refactor branch from d83b806 to 098103e Compare September 9, 2024 09:13
The Makefile simplified building Triton VM despite the cyclic dependency
in its build process. Now, the cyclic build dependency is broken. Cargo
is the recommended build tool for Triton VM.
@jan-ferdinand jan-ferdinand force-pushed the circuit_builder_refactor branch from 3da25ea to 278244a Compare September 9, 2024 13:35
Over the years, the terminology of “main” and “auxiliary” tables has
become dominant over our terminology of “base” and “extension” tables.

changelog: ignore
@jan-ferdinand jan-ferdinand merged commit f594167 into master Sep 10, 2024
4 checks passed
@jan-ferdinand jan-ferdinand deleted the circuit_builder_refactor branch September 10, 2024 13:24
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