-
Notifications
You must be signed in to change notification settings - Fork 123
Direct to APC trace gen #3452
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
base: main
Are you sure you want to change the base?
Direct to APC trace gen #3452
Conversation
|
Direct to APC Fibo: Direct to APC Keccak: |
|
Using the 10,000 Keccak pass:
Some thoughts:
Direct to APC numbers: Original numbers: Sample command run: |
|
|
Skip record only: Skip record + direct to APC for all Keccak chips (4 chips): Prior savings: New savings: |
|
The diff is showing a lot of unrelated things. Would be good to clean this up so it's clearer what the changes are here. |
…2LoadStoreAdapterAir, LoadSignExtendCoreAir<4, 8>
…racegen from powdr gpu trace gen entirely; all 13 chips work for reth prove-app with 10 APC
…enerate_proving_ctx_new
08f0c5f to
a351d89
Compare
3751367 to
7de4b0c
Compare
7de4b0c to
dfe2159
Compare
|
Another benchmarking pass after we minimized diffs in OVM. The following are all TOTAL tracegen numbers for 10K Keccak including both APC and non-APC, both record passing and tracegen per se:
So we somehow got even faster although I was expecting the reverse (because we removed many manual |
|
@qwang98 is this still going in? |
Powdr part ready for review (though OVM is not ready as I still need to consolidate APIs).
Depends on OVM branch: powdr-labs/openvm#50
Depends on Stark-Backend branch: powdr-labs/stark-backend#18