Skip to content

mono bn254 51-bit SIMD FMA multiplier#269

Merged
Bisht13 merged 21 commits into
mainfrom
xr/rne-mono
Feb 9, 2026
Merged

mono bn254 51-bit SIMD FMA multiplier#269
Bisht13 merged 21 commits into
mainfrom
xr/rne-mono

Conversation

@xrvdg
Copy link
Copy Markdown
Collaborator

@xrvdg xrvdg commented Feb 2, 2026

bn254 multiplier variant that uses SIMD round-to-nearest fused multiply add to compute a single multiplication. This variant is:

  • the fastest single multiplier on ARM cortex, but not as high throughput as the batching variants.
  • the fastest single multiplier on WASM and outperforms the batched variant batched bn254 51 bits RNE multiplier  #259 on WASM.

Merge after #259

@xrvdg xrvdg marked this pull request as draft February 2, 2026 11:18
@xrvdg xrvdg changed the title b51 FMA mono multiplier bn254 51-bit SIMD FMA mono multiplier Feb 3, 2026
@xrvdg xrvdg changed the title bn254 51-bit SIMD FMA mono multiplier mono bn254 51-bit SIMD FMA multiplier Feb 3, 2026
@xrvdg xrvdg marked this pull request as ready for review February 3, 2026 02:30
@xrvdg xrvdg requested a review from Bisht13 February 3, 2026 02:31
@Bisht13 Bisht13 requested a review from ocdbytes February 4, 2026 08:26
Copy link
Copy Markdown
Collaborator

@ocdbytes ocdbytes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Comment thread skyscraper/bn254-multiplier/src/rne/mono.rs Outdated
Comment thread skyscraper/bn254-multiplier/src/rne/mono.rs Outdated
Base automatically changed from xr/fmul to main February 6, 2026 11:15
@Bisht13 Bisht13 merged commit 26df97d into main Feb 9, 2026
4 of 5 checks passed
@Bisht13 Bisht13 deleted the xr/rne-mono branch February 9, 2026 09:46
dcbuild3r pushed a commit that referenced this pull request May 16, 2026
mono bn254 51-bit SIMD FMA multiplier
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.

3 participants