a16z / jolt

The simplest and most extensible zkVM. Fast and fully open source from a16z crypto and friends. ⚡
https://jolt.a16zcrypto.com
MIT License
619 stars 123 forks source link

Refactoring get_polys_r1cs to use references instead of cloning #307

Closed MatteoMer closed 4 months ago

MatteoMer commented 4 months ago

Solves #295

I had to refactor R1CSInputs a bit, to accept references

Doing benchmarks on my machine using cargo run --profile build-fast -p jolt-core trace --name sha3 --format chrome get_poly_r1cs gets a ~5x improvement (138ms to 38ms)