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

Dom/cache len #1

Draft
wants to merge 16 commits into
base: main
Choose a base branch
from
Draft

Dom/cache len #1

wants to merge 16 commits into from

Conversation

DomHenderson
Copy link

Adds a len method to the ProgramCache so that external crates can check its size

Checklist

  • Linked to Github Issue
  • Unit tests added
  • Integration tests added.
  • This change requires new documentation.
    • Documentation has been added/updated.

edg-l and others added 16 commits June 28, 2024 19:39
* Cleanup values.rs trying to make the code clearer

* rephrase

---------

Co-authored-by: Pedro Fontana <fontana.pedro93@gmail.com>
* Fix `snapshot_take` for arrays.

* Add test.

* Fix clippy issues.
* make nullable deep copy too

* format

* try

* fix

* clippy

* add test
…airo's u256 (lambdaclass#720)

* fix u256 fields swapped

* fix tests

* apply suggestion

* add test

* remove dbg

* fix gas
* Fix stuff.

* Implement `x86_64`-specific stuff.

* Fix debug utils print felt. Remove debug prints. Make JitValue print the felt value on `fmt::Debug` instead of its montgomery form limbs.

* Add test for the `Failed to deserialize param #1` bug (in arm64).

* Remove old todo comments.

* Add some docs.

* Add more docs.
* Simplify debug utils.

* Add auto breakpoint.

* Fix stuff.
…aclass#693)

* add CompilerError and fix BoundedInt range check

* fix clippy

* small fix

* Update src/error.rs

Co-authored-by: Mario Rugiero <mrugiero@gmail.com>

---------

Co-authored-by: Mario Rugiero <mrugiero@gmail.com>
Co-authored-by: Iñaki Garay <igarai@gmail.com>
Co-authored-by: Edgar <git@edgarluque.com>
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.

5 participants