<div dir="auto">LGTM and thank you. </div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Jan 9, 2020, 1:32 PM Daniel Sanders via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">dsanders updated this revision to Diff 237185.<br>
dsanders marked 2 inline comments as done.<br>
dsanders added a comment.<br>
<br>
Add a bit about asking someone to commit for you so that we can document<br>
providing the information needed for the Author property<br>
<br>
<br>
Repository:<br>
  rG LLVM Github Monorepo<br>
<br>
CHANGES SINCE LAST ACTION<br>
  <a href="https://reviews.llvm.org/D72468/new/" rel="noreferrer noreferrer" target="_blank">https://reviews.llvm.org/D72468/new/</a><br>
<br>
<a href="https://reviews.llvm.org/D72468" rel="noreferrer noreferrer" target="_blank">https://reviews.llvm.org/D72468</a><br>
<br>
Files:<br>
  llvm/docs/DeveloperPolicy.rst<br>
<br>
<br>
Index: llvm/docs/DeveloperPolicy.rst<br>
===================================================================<br>
--- llvm/docs/DeveloperPolicy.rst<br>
+++ llvm/docs/DeveloperPolicy.rst<br>
@@ -292,8 +292,15 @@<br>
<br>
 Below are some guidelines about the format of the message itself:<br>
<br>
-* Separate the commit message into title, body and, if you're not the original<br>
-  author, a "Patch by" attribution line (see below).<br>
+* Separate the commit message into title and body separated by a blank line.<br>
+<br>
+* If you're not the original author, ensure the 'Author' property of the commit is<br>
+  set to the original author and the 'Committer' property is set to yourself.<br>
+  You can use a command similar to<br>
+  ``git commit --amend --author="John Doe <<a href="mailto:jdoe@llvm.org" target="_blank" rel="noreferrer">jdoe@llvm.org</a>>`` to correct the<br>
+  author property if it is incorrect. See `Attribution of Changes`_ for more<br>
+  information including the method we used for attribution before the project<br>
+  migrated to git.<br>
<br>
 * The title should be concise. Because all commits are emailed to the list with<br>
   the first line as the subject, long titles are frowned upon.  Short titles<br>
@@ -314,11 +321,6 @@<br>
<br>
 * If the patch fixes a bug in bugzilla, please include the PR# in the message.<br>
<br>
-* `Attribution of Changes`_ should be in a separate line, after the end of<br>
-  the body, as simple as "Patch by John Doe.". This is how we officially<br>
-  handle attribution, and there are automated processes that rely on this<br>
-  format.<br>
-<br>
 * Text formatting and spelling should follow the same rules as documentation<br>
   and in-code comments, ex. capitalization, full stop, etc.<br>
<br>
@@ -340,6 +342,11 @@<br>
 quality patches.  If you would like commit access, please send an email to<br>
 `Chris <mailto:<a href="mailto:clattner@llvm.org" target="_blank" rel="noreferrer">clattner@llvm.org</a>>`_ with your GitHub username.<br>
<br>
+Prior to obtaining commit access, it is common practice to request that<br>
+someone with commit access commits on your behalf. When doing so, please<br>
+provide the name and email address you would like to use in the Author<br>
+property of the commit.<br>
+<br>
 Your first commit to a repository may require the autogenerated email to be<br>
 approved by a moderator of the mailing list.<br>
 This is normal and will be done when the mailing list owner has time.<br>
@@ -489,6 +496,11 @@<br>
 list, development list, or LLVM bug tracker component. If someone sends you<br>
 a patch privately, encourage them to submit it to the appropriate list first.<br>
<br>
+Our previous version control system (subversion) did not distinguish between the<br>
+author and the committer like git does. As such, older commits used a different<br>
+attribution mechanism. The previous method was to include "Patch by John Doe."<br>
+in a separate line of the commit message and there are automated processes that<br>
+rely on this format.<br>
<br>
 .. _IR backwards compatibility:<br>
<br>
<br>
<br>
</blockquote></div>