0xPolygonMiden / miden-vm

STARK-based virtual machine
MIT License
615 stars 152 forks source link

Optimize push 2 #1253

Closed hackaugusto closed 6 months ago

hackaugusto commented 6 months ago

Describe your changes

This encodes the same value a little bit more effectively:

Checklist before requesting a review

hackaugusto commented 6 months ago

There are some CI errors, seems the tests are checking the clock count. I will fix this if the PR gets a go ahead.

hackaugusto commented 6 months ago

but also not sure if it is worth the effort as we'd also need to update cycle counts for downstream operations which use append_pow2_op

closing this