CONTRIBUTING.md: elaborate LLM policy and add copy of LLVM's LLM policy#193
Conversation
I copied the LLVM policy wholesale. I considered search-and-replacing LLVM with "rust-simplicity" or "Simplicity" or something but this seemed like a lot of tedious work that would need to be repeated slightly-differently for every repo, and which wouldn't add much value.
apoelstra
force-pushed
the
2026-01/ai-policy
branch
from
00d9fa0 to
1943fe9
Compare
| This PR appears to be extractive, and requires additional justification for | ||
| why it is valuable enough to the project for us to review it. Please see | ||
| our developer policy on AI-generated contributions: | ||
| http://llvm.org/docs/AIToolPolicy.html |
There was a problem hiding this comment.
Heh, interesting. If you are feeling motivated, you could file an issue on github.com/rnk/llvm-project -- this comes from https://raw.githubusercontent.com/rnk/llvm-project/refs/heads/tool-policy/llvm/docs/AIToolPolicy.md and I didn't want to modify it because then I'd have to add a header describing my modifications.
There was a problem hiding this comment.
Left a comment here: llvm/llvm-project#154441 (comment)
There was a problem hiding this comment.
This link is supposed to be linking to this document itself, and should refer to the SimpliciytHL repo.
There was a problem hiding this comment.
This issue wasn't resolved.
There was a problem hiding this comment.
We are not going to modify the LLVM policy. The link is dead in their text. Therefore the bug is on their end.
There was a problem hiding this comment.
You already did change the policy:
- repository or organization.
+ repository or organization. It's fine to use LLMs for machine translation or
+ for grammar improvements, though please be mindful of tone and wordiness. We
+ would much rather read poor English than ChatGPT-style English.
The reason their link is dead is because llvm/llvm-project#154441 isn't merged yet!
Look at the name of the file in the link: AIToolPolicy.html, now look at the name of the file in llvm/llvm-project#154441 that you copied: AIToolPolicy.md. The link in the text that people are supposed to copy is a link back to the file you copied (and modified!) itself. We need to folks to link to this policy we just merged.
There was a problem hiding this comment.
You already did change the policy:
That text is in CONTRIBUTING.md, not the document which is taken from LLVM.
uncomputable
left a comment
uncomputable
left a comment
There was a problem hiding this comment.
ACK 1943fe9 The policy looks good besides one dead link
|
Added a commit clarifying that it's fine to use LLMs for machine translation or grammar improvements. |
|
ACK 3ef0434 |
5 participants
I copied the LLVM policy wholesale.
I considered search-and-replacing LLVM with "rust-simplicity" or "Simplicity" or something but this seemed like a lot of tedious work that would need to be repeated slightly-differently for every repo, and which wouldn't add much value.