-
Notifications
You must be signed in to change notification settings - Fork 36
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
- Placing the Poseidon `state` columns after the `partial_sbox` column instead of before it causes them to line up with vast stretch of free space, enabling the pad-and-add region to be layed out there. - Using the `Region::assign_advice_from_constant` API to initialise the Poseidon state removes fixed-column contention between that region and fixed-base scalar multiplication, enabling it to also be layed out within the free space. - If zcash/halo2#334 were implemented then this region would disappear. - The overflow check in variable-base scalar mul is also moved into the columns with free space.
- Loading branch information
Showing
3 changed files
with
11 additions
and
14 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters