<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Apr 23, 2014 at 1:10 PM, David Blaikie <span dir="ltr"><<a href="mailto:dblaikie@gmail.com" target="_blank" class="cremed">dblaikie@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div class="">On Wed, Apr 23, 2014 at 8:21 AM, Diego Novillo <<a href="mailto:dnovillo@google.com" class="cremed">dnovillo@google.com</a>> wrote:<br>

> Author: dnovillo<br>
> Date: Wed Apr 23 10:21:23 2014<br>
> New Revision: 206997<br>
><br>
> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=206997&view=rev" target="_blank" class="cremed">http://llvm.org/viewvc/llvm-project?rev=206997&view=rev</a><br>
> Log:<br>
> Review feedback.<br>
<br>
</div>It's useful to mention what the actual change is (even if it's<br>
something simple like "correct grammar") on the first line of the<br>
commit message.<br>
<br>
Then I usually put "Code review feedback from rXXXX by John Smith"<br>
after a blank line, so people can search for the revision number and<br>
find the original commit/code review feedback/context/etc.<br></blockquote><div><br></div><div>As I said in <a href="http://lists.cs.uiuc.edu/pipermail/cfe-commits/Week-of-Mon-20140421/103806.html">http://lists.cs.uiuc.edu/pipermail/cfe-commits/Week-of-Mon-20140421/103806.html</a>, this was me not rebasing my changes before the commit.  All the actual conversation happened in the thread.  Phabricator just makes it hard to mix the two things, which causes issues like this one when you're not paying enough attention.</div>
<div><br></div><div>Diego.</div></div></div></div>