Docs are out of date now that we have separate repositories for LLVM,
Clang, etc.
Details
Diff Detail
- Repository
- rL LLVM
Event Timeline
Am I correct in thinking that as long as the repository field is filled out, there should never be a reason to explicitly add llvm-commits or cfe-commits to the subscriber list? If so it would probably make sense to remove the note "subscribe mailing lists that you want to be included in the review. If your patch is for LLVM, add llvm-commits as a Subscriber; if your patch is for Clang, add cfe-commits." Instead, you could expand the bullet point on the repository field to say "Entering the correct repository will ensure that the appropriate mailing list (llvm-commits or cfe-commits) is automatically subscribed to patch patch updates.
there should never be a reason to explicitly add llvm-commits or cfe-commits to the subscriber list?
Good catch, fixed.