Skip to content
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.

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

Make 7x7 non_zero count and zig49 a single array #68

Closed
wants to merge 1 commit into from
Closed

Conversation

mcroomp
Copy link
Collaborator

@mcroomp mcroomp commented Apr 23, 2024

  • Incrementally advance through array as we update
  • Align counts at 256 byte boundary (types are already close to that)
  • Split out zero prior_bit_len since this is many times sequentially accessed, it also makes the counts just under 256 bytes (as opposed to 0x106 bytes)

@Melirius
Copy link
Collaborator

Hmm, on my machine it does not improve performance.

@mcroomp
Copy link
Collaborator Author

mcroomp commented Apr 26, 2024

Not sure why it's slower... oh well. Maybe I missed something in the way code generation works.

@mcroomp mcroomp closed this Apr 26, 2024
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