<div dir="ltr"><div dir="ltr">I made a mistake when committing r350579 -- I saved the commit message into a file but I used the wrong file. I will revert r350579 and recommit.</div></div><br><div class="gmail_quote"><div dir="ltr">On Tue, Jan 8, 2019 at 1:48 PM Teresa Johnson via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">tejohnson added a comment.<br>
<br>
In D56327#1350006 <<a href="https://reviews.llvm.org/D56327#1350006" rel="noreferrer" target="_blank">https://reviews.llvm.org/D56327#1350006</a>>, @kristina wrote:<br>
<br>
> Differential Review is mostly for pre-commit review, add in "Differential Revision: <a href="https://reviews.llvm.org/DXXXXX" rel="noreferrer" target="_blank">https://reviews.llvm.org/DXXXXX</a>" to the commit message and it will automatically link the commit and diff and close it. As an alternative you can do that manually, It seems bigger patch has already been committed in rL350579 <<a href="https://reviews.llvm.org/rL350579" rel="noreferrer" target="_blank">https://reviews.llvm.org/rL350579</a>> (without review)<br>
<br>
<br>
@xur  - something weird happened with your commits. The one mentioned by @kristina  above (r350579) was actually D56342 <<a href="https://reviews.llvm.org/D56342" rel="noreferrer" target="_blank">https://reviews.llvm.org/D56342</a>> ([llvm-profdata] add value-cutoff functionality to show command) which was reviewed, but somehow got the same title and commit message as this patch. Not sure if there is a way to go back and fix that in the commit log...if not I'd recommend reverting it and recommitting with the correct commit log and title.<br>
<br>
Also, as @kristina  mentions, if your commit log contains a message like "Differential Revision: <a href="https://reviews.llvm.org/DXXXXX" rel="noreferrer" target="_blank">https://reviews.llvm.org/DXXXXX</a>" it would link to the patch on phab.<br>
<br>
> and this was pending review rL350442 <<a href="https://reviews.llvm.org/rL350442" rel="noreferrer" target="_blank">https://reviews.llvm.org/rL350442</a>>. I've linked those in but if you say it's been fixed in the committed version and this is a followup, can you rebase the patch on the current trunk? For post-commit reviews Audit may be a better place if you need that.<br>
<br>
<br>
<br>
<br>
CHANGES SINCE LAST ACTION<br>
  <a href="https://reviews.llvm.org/D56327/new/" rel="noreferrer" target="_blank">https://reviews.llvm.org/D56327/new/</a><br>
<br>
<a href="https://reviews.llvm.org/D56327" rel="noreferrer" target="_blank">https://reviews.llvm.org/D56327</a><br>
<br>
<br>
<br>
</blockquote></div>