Skip to content

Conversation

Golovanov399
Copy link
Contributor

Towards INT-4117.

This makes some fields and functions more visible for the GPU to be able to use them.

@Golovanov399 Golovanov399 changed the title Add pubs needed for gpu feat: Add pubs needed for gpu Jul 14, 2025
Copy link

codspeed-hq bot commented Jul 14, 2025

CodSpeed WallTime Performance Report

Merging #1860 will not alter performance

Comparing feat/access-adapters-more-pubs (be74379) with feat/new-execution (a6be3df)

⚠️ Unknown Walltime execution environment detected

Using the Walltime instrument on standard Hosted Runners will lead to inconsistent data.

For the most accurate results, we recommend using CodSpeed Macro Runners: bare-metal machines fine-tuned for performance measurement consistency.

Summary

✅ 20 untouched benchmarks

Copy link

group app.proof_time_ms app.cycles app.cells_used leaf.proof_time_ms leaf.cycles leaf.cells_used
verify_fibair (+61 [+4.9%]) 1,307 322,699 17,271,190 - - -
fibonacci (+32 [+1.4%]) 2,369 1,500,277 50,589,231 - - -
regex (-26 [-0.4%]) 6,962 4,165,432 166,449,586 - - -
ecrecover (-6 [-0.4%]) 1,354 137,283 8,179,549 - - -
pairing (-5 [-0.1%]) 3,980 1,862,964 102,530,853 - - -

Commit: be74379

Benchmark Workflow

@Golovanov399 Golovanov399 merged commit 48c80d7 into feat/new-execution Jul 14, 2025
34 of 35 checks passed
@Golovanov399 Golovanov399 deleted the feat/access-adapters-more-pubs branch July 14, 2025 14:21
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

Successfully merging this pull request may close these issues.

2 participants