0xPolygonZero / plonky2

Apache License 2.0
763 stars 285 forks source link

Optimize reduce_polys_base #1586

Closed Daniel-Aaron-Bloom closed 5 months ago

Daniel-Aaron-Bloom commented 5 months ago

Updated version of #1034, because apparently removing the collect (as #1571 tried to) significantly worsens performance.

Daniel-Aaron-Bloom commented 5 months ago

Bummer.

On my end, cargo run --release --example bench_recursion -- -vv --size=18 seems to show no real difference between any of them here so I don't disagree.

Feel free to close all related PRs.