search

0xPolygonZero / plonky2

Apache License 2.0
757 stars 283 forks source link

Use logUp for CTLs #1398

Closed hratoanina closed 9 months ago

hratoanina commented 9 months ago

This PR changes the CTL argument for logUp. This doesn't bring any benefits on its own, but it allows further optimizations: degree 2 CTL filters and CTL bundling.

sonarcloud[bot] commented 9 months ago

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

hratoanina commented 9 months ago

I reverted back to the previous order, with the CTLs before range checks. In the future it would be nicer to have a dedicated section for logUp anyway.