This PR changes sym_n to keep a single AxEffects object which is updated (rather than replaced) each step.
Currently, this significantly slows down symbolic simulation, causing timeouts when simulation more than ~60 steps of SHA512, and thus breaking a lot of proofs.
Testing:
What tests have been run? Did make all succeed for your changes? Was
conformance testing successful on an Aarch64 machine?
License:
By submitting this pull request, I confirm that my contribution is
made under the terms of the Apache 2.0 license.
Description:
Stacked on
This PR changes
sym_n
to keep a singleAxEffects
object which is updated (rather than replaced) each step.Currently, this significantly slows down symbolic simulation, causing timeouts when simulation more than ~60 steps of SHA512, and thus breaking a lot of proofs.
Testing:
What tests have been run? Did
make all
succeed for your changes? Was conformance testing successful on an Aarch64 machine?License:
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.