Skip to content

Conversation

@flaub
Copy link
Member

@flaub flaub commented Nov 13, 2024

This needs to land first: risc0/risc0#2504

@flaub flaub requested a review from tzerrell November 13, 2024 08:21
@flaub flaub self-assigned this Nov 13, 2024
@flaub flaub marked this pull request as draft November 13, 2024 08:33
Copy link

@tzerrell tzerrell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome! Thanks also for the cleanup of how this managed num-bigint-dig!

@flaub flaub marked this pull request as ready for review November 13, 2024 18:45
@flaub flaub merged commit fa372e9 into risc0 Nov 13, 2024
@flaub flaub deleted the flaub/bigint2 branch November 13, 2024 19:19
flaub added a commit to risc0/risc0 that referenced this pull request Nov 13, 2024
The following need to land first:

* #2504
* risc0/RustCrypto-RSA#3

Closes ZKVM-638
austinabell added a commit that referenced this pull request Dec 13, 2024
Use the new RSA extern (#2)

---------

Co-authored-by: Frank Laub <github@frank.laub.io>
Use risc0-bigint2 (#3)

* Use risc0-bigint2

* Use num-bigint-dig feature

* Update lockfile

* Update ref

* Update ref

* Update git ref
Update bigint2 impl with 4096 bit support (#4)

* update acceleration to use latest version of bigint2 (with 4096 bit support)

* bump version

* bump to 1.2
austinabell added a commit that referenced this pull request Dec 13, 2024
* Add Zirgen-based acceleration (#1)

Use the new RSA extern (#2)

---------

Co-authored-by: Frank Laub <github@frank.laub.io>
Use risc0-bigint2 (#3)

* Use risc0-bigint2

* Use num-bigint-dig feature

* Update lockfile

* Update ref

* Update ref

* Update git ref
Update bigint2 impl with 4096 bit support (#4)

* update acceleration to use latest version of bigint2 (with 4096 bit support)

* bump version

* bump to 1.2

* gate prop tests behind cfg to enable cargo risczero test
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.

3 participants