This is an archive of the discontinued LLVM Phabricator instance.

Expand the LLVM Developer Policy to include new sections on adding a project to the LLVM Monorepo, and a second about the LLVM Incubator projects.
ClosedPublic

Authored by lattner on Jul 5 2020, 1:41 PM.

Diff Detail

Event Timeline

lattner created this revision.Jul 5 2020, 1:41 PM
Herald added a project: Restricted Project. · View Herald TranscriptJul 5 2020, 1:41 PM

This is the Google doc translated to RST format.

stellaraccident accepted this revision.Jul 5 2020, 1:49 PM
This revision is now accepted and ready to land.Jul 5 2020, 1:49 PM

I suggested a few minor rewrites that bugged me more on a second reading.

llvm/docs/DeveloperPolicy.rst
534

complete sentence?

546

"We want... " is awkward. How about "This check should first be enabled as a 'soft-error' which can be turned into a warning using a CMake flag."

677–678

Seems a little anthropomorphic. how about " and make it easier for subcommunities to collaborate"

718

"carveout" may not be something people are familiar with. Maybe "teams that already have permission to make contributions..."

Thanks! I'm going to let this simmer for a few days to give folks in the community time to react / improve.

lattner marked 4 inline comments as done.Jul 5 2020, 2:29 PM

Thank you for the feedback Stephen!

llvm/docs/DeveloperPolicy.rst
534

<snark>Oh I see what you are doing here. I just move a section from one place to the other and now I have to make it actually make sense. That could never have just recently happened to you ;-)

lattner updated this revision to Diff 275576.Jul 5 2020, 2:30 PM

Changes suggested by Stephen Neuendorffer

stephenneuendorffer marked an inline comment as done.Jul 6 2020, 9:00 AM
stephenneuendorffer added inline comments.
llvm/docs/DeveloperPolicy.rst
534

No good deed goes unpunished. :)

This revision was automatically updated to reflect the committed changes.