Skip to content

Conversation

@apoelstra
Copy link
Contributor

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 apoelstra requested a review from imaginator as a code owner January 2, 2026 17:41
Copy link
Collaborator

@delta1 delta1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK 00d9fa0

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.
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
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

1943fe9 This is a dead link

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Left a comment here: llvm/llvm-project#154441 (comment)

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This link is supposed to be linking to this document itself, and should refer to the SimpliciytHL repo.

Copy link
Collaborator

@uncomputable uncomputable left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK 1943fe9 The policy looks good besides one dead link

Copy link
Collaborator

@delta1 delta1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK 1943fe9

Copy link
Collaborator

@KyrylR KyrylR left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK 1943fe9

@apoelstra
Copy link
Contributor Author

Added a commit clarifying that it's fine to use LLMs for machine translation or grammar improvements.

Copy link
Collaborator

@delta1 delta1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK 3ef0434

Copy link
Contributor Author

@apoelstra apoelstra left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

On 3ef0434 successfully ran local tests

@apoelstra apoelstra merged commit 4ad9d38 into BlockstreamResearch:master Jan 5, 2026
10 checks passed
@KyrylR
Copy link
Collaborator

KyrylR commented Jan 6, 2026

ACK 3ef0434

@apoelstra apoelstra deleted the 2026-01/ai-policy branch January 6, 2026 15: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.

5 participants