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

Credit "the paper" from which the PCLMULQDQ specialization came from #40

Merged
merged 1 commit into from
May 20, 2024

Conversation

AlexTMjugador
Copy link
Contributor

While "Fast CRC computation for generic polynomials using PCLMULQDQ instruction" is a well-known paper in the realm of specialized CRC and compression algorithms for the x86 architecture that has seen widespread implementation, not everyone is aware of it, and it's nevertheless a good practice to cite the sources ideas and knowledge come from.

Let's add a comment to the pclmulqdq module citing the whitepaper, including a link to a mirror hosting its PDF, for easy access by prospective contributors and other interested parties.

While "Fast CRC computation for generic polynomials using PCLMULQDQ
instruction" is a well-known paper in the realm of specialized CRC and
compression algorithms for the x86 architecture that has seen widespread
implementation, not everyone is aware of it, and it's nevertheless a
good practice to cite the sources ideas and knowledge come from.

Let's add a comment to the `pclmulqdq` module citing the whitepaper,
including a link to a mirror hosting its PDF, for easy access by
prospective contributors and other interested parties.
@AlexTMjugador
Copy link
Contributor Author

This PR fixes #34 😄

@srijs srijs merged commit 44284e4 into srijs:master May 20, 2024
10 checks passed
@srijs
Copy link
Owner

srijs commented May 20, 2024

Thanks!

@srijs srijs mentioned this pull request May 20, 2024
@AlexTMjugador AlexTMjugador deleted the docs/the-paper branch May 20, 2024 22:16
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.

None yet

2 participants