Skip to content

ZKVM-1381: fix: Remove acceleration of ct_inverse_mod_n#7

Merged
flaub merged 1 commit intorisc0from
noinv
May 13, 2025
Merged

ZKVM-1381: fix: Remove acceleration of ct_inverse_mod_n#7
flaub merged 1 commit intorisc0from
noinv

Conversation

@hashcashier
Copy link

This fixes the c-kzg verification flow.

@hashcashier hashcashier changed the title fix(ct_inverse_mod_n): Remove acceleration of ct_inverse_mod_n fix: Remove acceleration of ct_inverse_mod_n May 13, 2025
@github-actions github-actions bot changed the title fix: Remove acceleration of ct_inverse_mod_n ZKVM-1380: fix(ct_inverse_mod_n): Remove acceleration of ct_inverse_mod_n May 13, 2025
@hashcashier hashcashier self-assigned this May 13, 2025
@github-actions github-actions bot changed the title ZKVM-1380: fix(ct_inverse_mod_n): Remove acceleration of ct_inverse_mod_n ZKVM-1381: fix: Remove acceleration of ct_inverse_mod_n May 13, 2025
@hashcashier hashcashier requested review from ec2, flaub and jbruestle May 13, 2025 09:09
@flaub flaub merged commit 8922ef3 into risc0 May 13, 2025
5 checks passed
@flaub flaub deleted the noinv branch May 13, 2025 19:12
Wollac added a commit that referenced this pull request May 14, 2025
flaub pushed a commit that referenced this pull request May 20, 2025
* Revert "don't accelerate modinv (#7)"

This reverts commit 8922ef3.

* set ret to zero

* compute inv directly

* fix sizeof
Wollac pushed a commit that referenced this pull request Jul 23, 2025
Wollac added a commit that referenced this pull request Jul 23, 2025
* Revert "don't accelerate modinv (#7)"

This reverts commit 8922ef3.

* set ret to zero

* compute inv directly

* fix sizeof
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.

2 participants