Skip to content

Extend the small inverse pow5 offset table #237

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
keith-packard wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Extend the small inverse pow5 offset table #237

keith-packard wants to merge 1 commit into from

Conversation

@keith-packard
Copy link

@keith-packard keith-packard commented Apr 12, 2025 •
edited
Loading

The maximum value for the invPow5 function is 342, which means the offset table needs 342 * 2 = 684 bits or > 21 32-bit values. Add the missing values by computing them based off the full table values.
@keith-packard keith-packard force-pushed the extend-small-inv-table branch from 12a63e0 to e20b956 Compare April 12, 2025 22:37
@keith-packard
Extend the small inverse pow5 offset table
78e3e5b 
The maximum value for the invPow5 function is 342, which means the
offset table needs 342 * 2 = 684 bits or > 21 32-bit values. Add the
missing values by computing them based off the full table values.

Signed-off-by: Keith Packard <keithp@keithp.com>
@keith-packard keith-packard force-pushed the extend-small-inv-table branch from e20b956 to 78e3e5b Compare April 12, 2025 22:41
@DarinM223 DarinM223 mentioned this pull request May 12, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

1 participant

@keith-packard