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

Replace c_binds with c_bind_new in barretenberg #3076

Open
kevaundray opened this issue Oct 26, 2023 · 0 comments
Open

Replace c_binds with c_bind_new in barretenberg #3076

kevaundray opened this issue Oct 26, 2023 · 0 comments
Labels
C-barretenberg Component: barretenberg cryptography library

Comments

@kevaundray
Copy link
Contributor

Problem

We currently have duplicated code related to c_bind and c_bind_new. Ideally we remove the c_bind files and only use c_bind_new.

The issue right now is that circuits relies on c_bind, and so we cannot remove it right now. Noir's acvm_backend_wasm also uses c_bind.

Solution

Either we have circuits and Noir calling the methods in c_bind_new and or we have those components re-implement the necessary logic.

@kevaundray kevaundray added the C-barretenberg Component: barretenberg cryptography library label Oct 26, 2023
@github-project-automation github-project-automation bot moved this to Todo in A3 Oct 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-barretenberg Component: barretenberg cryptography library
Projects
Status: Todo
Development

No branches or pull requests

1 participant