Skip to content

Conversation

@abadams
Copy link
Member

@abadams abadams commented Jan 5, 2022

This restricts the new simplifier rules to unsigned only.

By pulling constant additions outside of quadratics, we can shave off a
few add instructions in the inner loop for random number generation,
which uses a quadratic modulo 2^32

I also removed the !overflows predicates, because rules already fail to
match if a fold overflows.

New rules formally verified.
@abadams abadams requested a review from steven-johnson January 5, 2022 19:19
@steven-johnson
Copy link
Contributor

Failure is unrelated

@steven-johnson steven-johnson merged commit b8a711a into master Jan 7, 2022
@steven-johnson steven-johnson deleted the abadams/low_bit_noise branch January 7, 2022 18:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants