Overview of changes --------------------- - Use CBIND macro for merge_rollup__sim and root_rollup__sim - Data structure of the inputs/outputs of these function need to be reconciled between C++ and TS (member names need to match as circuits.gen.ts will require reference to TS data structure generated from the C++ counterparts. In addition, migration from array to TS tuple is required.) - Activate new code path in the upper layer (remove wrapper functions in rollup_wasm_wrapper and directly use mergeRollupSim and rootRollupSim from circuits.gen.ts). Resolves #2116 Resolves #2117 # Checklist: Remove the checklist to signal you've completed it. Enable auto-merge if the PR is ready to merge. - [x] If the pull request requires a cryptography review (e.g. cryptographic algorithm implementations) I have added the 'crypto' tag. - [x] I have reviewed my diff in github, line by line and removed unexpected formatting changes, testing logs, or commented-out code. - [x] Every change is related to the PR description. - [x] I have [linked](https://docs.github.com/en/issues/tracking-your-work-with-issues/linking-a-pull-request-to-an-issue) this pull request to relevant issues (if any exist).

refactor: Root Rollup - migrate cbind routines to mspack #2117

Description

Metadata

Metadata

Assignees

Labels

Type

Fields

Projects

Milestone

Relationships

Development

Issue actions