[libc-commits] [PATCH] D136487: [libc.search] [PATCH 1/6] add wyhash as a header library

[Chandler Carruth via Phabricator via libc-commits]

chandlerc added a comment.

Not active really here, but wanted to point out --

wyhash is not under the public domain AFAIK. The repository is here:
https://github.com/wangyi-fudan/wyhash

The license is the so-called "unlicense" which you can read about here:
https://softwareengineering.stackexchange.com/questions/147111/what-is-wrong-with-the-unlicense

In general, contributions to LLVM projects need to follow the basic rules in the developer policy, including that it is licensed under the LLVM license:
https://llvm.org/docs/DeveloperPolicy.html#copyright-license-and-patents:~:text=When%20you%20contribute%20code%20to%20the%20LLVM%20project%2C%20you%20license%20it%20under%20these%20terms.

That policy includes specific instructions for suggesting the inclusion of code under other licenses:
https://llvm.org/docs/DeveloperPolicy.html#copyright-license-and-patents:~:text=In%20certain%20circumstances,requesting%20a%20review.

Snippet for ease:

> In certain circumstances, code licensed under other licenses can be added to the codebase. However, this may only be done with approval of the LLVM Foundation Board of Directors, and contributors should plan for the approval process to take at least 4-6 weeks. If you would like to contribute code under a different license, please create a Phabricator review with the code you want to contribute and email board at llvm.org requesting a review.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D136487/new/

https://reviews.llvm.org/D136487