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

Port Goldilock from Plonky2 with proper field arithmetics wrapper #483

Open
hero78119 opened this issue Oct 28, 2024 · 2 comments
Open

Port Goldilock from Plonky2 with proper field arithmetics wrapper #483

hero78119 opened this issue Oct 28, 2024 · 2 comments

Comments

@hero78119
Copy link
Collaborator

hero78119 commented Oct 28, 2024

As current Goldilock repo is out from personal repo from patching repo both lack of following up activities and probably no audit.

One possible solution is to switch to Plonky2 Goldilocks with more mature optimisation, e.g. avx

@matthiasgoergens
Copy link
Collaborator

matthiasgoergens commented Oct 28, 2024

Plonky2 has pretty decent code overall. Keep in mind that when I used plonky2 in the past, I did not see any benefit from avx (and the plonky2 authors weren't surprised). That might be because it's already pretty well optimised even without avx, or because they missed some avx optimisations. I'm not sure.

0xPolygonZero/plonky2#1621 might be interesting to watch!

@matthiasgoergens
Copy link
Collaborator

Until we move to plonky2, we can at least use a scroll-owned Goldilocks repo. See #459

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

No branches or pull requests

2 participants