Fix profile & analyze - Githubissues

ethereum / c-kzg-4844

A minimal implementation of the Polynomial Commitments API for EIP-4844 and EIP-7594, written in C.

Apache License 2.0

112 stars 105 forks source link

Fix profile & analyze #473

Closed jtraglia closed 1 month ago

jtraglia commented 1 month ago

So make profile and make analyze have been broken since the file split. This PR makes the following changes:

Run analyzer on each source file individually. Doesn't work when bundled together.
Fix some allocation nits in the tests. There were some potential leaks that the analyzer pointed out.
- I'm using more stack variables now. They are big, but not big enough to cause problems.
- This includes storing the result & doing c_kzg_free before doing asserts.
Fix commitments issue in one of the profiler functions.

~~To fix the broken test, please merge #474 first.~~