Remove some CPU cycles - Githubissues

0xPolygonZero / plonky2

Apache License 2.0

755 stars 278 forks source link

Closed Nashtare closed 7 months ago

Nashtare commented 7 months ago

Removes some CPU cycles:

all calls to %mload_packing can be directly replaced to MLOAD_32BYTES, no need for hook + jump
Similarly for %mstore_unpacking, we can sometimes just call directly the variant if the size is known which skips going through the jumptable
tweaked some very large stack operation in hash.asm, to reduce the needless overhead

Removes 1050 CPU cycles on ERC20 (much more with large MPTs)

sonarcloud[bot] commented 7 months ago

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No data about Coverage
No data about Duplication