Skip to content

[feat] use lookup tables for u16 assignment#2

Merged
zhenfeizhang merged 4 commits into
0.3.1-derive-serdefrom
impl-lookups
May 24, 2023
Merged

[feat] use lookup tables for u16 assignment#2
zhenfeizhang merged 4 commits into
0.3.1-derive-serdefrom
impl-lookups

Conversation

@zhenfeizhang

@zhenfeizhang zhenfeizhang commented May 22, 2023

Copy link
Copy Markdown

Without "bn256-table"

z@ubuntu:~/Desktop/0220/halo2curves$ cargo test --release bench_fr_from_u16 --features=print-trace  -- --nocapture 
Start:   generate 1000000 Bn256 scalar field elements
End:     generate 1000000 Bn256 scalar field elements ..............................38.108ms

Start:   generate 10000000 Bn256 scalar field elements
End:     generate 10000000 Bn256 scalar field elements .............................268.232ms

With "bn256-table"

z@ubuntu:~/Desktop/0220/halo2curves$ cargo test --release bench_fr_from_u16 --features=print-trace,bn256-table  -- --nocapture
Start:   generate 1000000 Bn256 scalar field elements
End:     generate 1000000 Bn256 scalar field elements ..............................12.365ms

Start:   generate 10000000 Bn256 scalar field elements
End:     generate 10000000 Bn256 scalar field elements .............................84.353ms

@zhenfeizhang zhenfeizhang merged commit c763162 into 0.3.1-derive-serde May 24, 2023
zhenfeizhang added a commit that referenced this pull request May 25, 2023
lightsing pushed a commit to lightsing/halo2curves that referenced this pull request Jan 30, 2024
* Bernstein yang modular multiplicative inverter (scroll-tech#2)

* rename similar to privacy-ethereum#95

---------

Co-authored-by: Aleksei Vambol <77882392+AlekseiVambol@users.noreply.github.com>
z2trillion pushed a commit that referenced this pull request Feb 22, 2024
Implement `ff::PrimeField::to_repr` for `bn256::Fq2`
Velaciela pushed a commit that referenced this pull request Aug 14, 2024
* Bernstein yang modular multiplicative inverter (#2)

* rename similar to privacy-ethereum#95

---------

Co-authored-by: Aleksei Vambol <77882392+AlekseiVambol@users.noreply.github.com>
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.

1 participant