Skip to content

chore: Document RAM gate cost optimization opportunity#21426

Merged
notnotraju merged 1 commit into
merge-train/barretenbergfrom
rk/optimize-ram-gate-count-comment
Mar 12, 2026
Merged

chore: Document RAM gate cost optimization opportunity#21426
notnotraju merged 1 commit into
merge-train/barretenbergfrom
rk/optimize-ram-gate-count-comment

Conversation

@notnotraju

@notnotraju notnotraju commented Mar 12, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Adds a comment above process_RAM_array documenting the current ~3.25 gate cost per RAM interaction and the potential optimization to reduce it to ~2.25 gates by deferring memory gate creation to circuit finalisation and embedding the timestamp relation directly.

Just documenting 522

@notnotraju notnotraju force-pushed the rk/optimize-ram-gate-count-comment branch from d139121 to 7ad6a64 Compare March 12, 2026 14:37
@notnotraju notnotraju force-pushed the rk/optimize-ram-gate-count-comment branch from 7ad6a64 to 6e75dad Compare March 12, 2026 14:40
@notnotraju notnotraju requested a review from iakovenkos March 12, 2026 14:41
@notnotraju notnotraju enabled auto-merge (squash) March 12, 2026 14:43
@notnotraju notnotraju merged commit 6303714 into merge-train/barretenberg Mar 12, 2026
10 checks passed
@notnotraju notnotraju deleted the rk/optimize-ram-gate-count-comment branch March 12, 2026 14:54
github-merge-queue Bot pushed a commit that referenced this pull request Mar 12, 2026
BEGIN_COMMIT_OVERRIDE
chore: unify tests in ecc/curves and ecc/groups (#21392)
chore: `ecc/fields` ASM audit (#20892)
chore: audit the `ecc/field` extension field (#21409)
chore: responding to external IPA audit (#20334)
feat: committed sumcheck for batched Hiding Translator (#21376)
chore: documentation fixes for ECCVM after first external audit (#20348)
chore: Document RAM gate cost optimization opportunity (#21426)
refactor: rename MSGPACK_FIELDS to SERIALIZATION_FIELDS (#21175)
END_COMMIT_OVERRIDE
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