<div dir="ltr">





<p class="inbox-inbox-p1"><span class="inbox-inbox-s1">Fix here: <a href="https://reviews.llvm.org/D41956">https://reviews.llvm.org/D41956</a></span></p></div><br><div class="gmail_quote"><div dir="ltr">On Thu, Jan 11, 2018 at 11:48 AM Ben Hamilton <<a href="mailto:benhamilton@google.com">benhamilton@google.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Whoops! Thanks for catching that, fixing now.</div><br><div class="gmail_quote"><div dir="ltr">On Thu, Jan 11, 2018 at 11:35 AM Robinson, Paul <<a href="mailto:paul.robinson@sony.com" target="_blank">paul.robinson@sony.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
> -----Original Message-----<br>
> From: llvm-commits [mailto:<a href="mailto:llvm-commits-bounces@lists.llvm.org" target="_blank">llvm-commits-bounces@lists.llvm.org</a>] On Behalf<br>
> Of Ben Hamilton via llvm-commits<br>
> Sent: Thursday, January 11, 2018 8:30 AM<br>
> To: <a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a><br>
> Subject: [llvm] r322290 - [docs] Update Phabricator docs about setting<br>
> repository for diffs uploaded via web<br>
><br>
> Author: benhamilton<br>
> Date: Thu Jan 11 08:30:08 2018<br>
> New Revision: 322290<br>
><br>
> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=322290&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project?rev=322290&view=rev</a><br>
> Log:<br>
> [docs] Update Phabricator docs about setting repository for diffs uploaded<br>
> via web<br>
><br>
> Summary:<br>
> Docs are out of date now that we have separate repositories for LLVM,<br>
> Clang, etc.<br>
><br>
> Reviewers: asb<br>
><br>
> Reviewed By: asb<br>
><br>
> Differential Revision: <a href="https://reviews.llvm.org/D41919" rel="noreferrer" target="_blank">https://reviews.llvm.org/D41919</a><br>
><br>
> Modified:<br>
>     llvm/trunk/docs/Phabricator.rst<br>
><br>
> Modified: llvm/trunk/docs/Phabricator.rst<br>
> URL: <a href="http://llvm.org/viewvc/llvm-" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-</a><br>
> project/llvm/trunk/docs/Phabricator.rst?rev=322290&r1=322289&r2=322290&vie<br>
> w=diff<br>
> ==========================================================================<br>
> ====<br>
> --- llvm/trunk/docs/Phabricator.rst (original)<br>
> +++ llvm/trunk/docs/Phabricator.rst Thu Jan 11 08:30:08 2018<br>
> @@ -65,14 +65,14 @@ To upload a new patch:<br>
>  * Click *Differential*.<br>
>  * Click *+ Create Diff*.<br>
>  * Paste the text diff or browse to the patch file. Click *Create Diff*.<br>
> -* Leave the Repository field blank.<br>
> +* In the Repository field, enter the name of the project (LLVM, Clang,<br>
> +  etc.) to which the review should be sent.<br>
>  * Leave the drop down on *Create a new Revision...* and click *Continue*.<br>
>  * Enter a descriptive title and summary.  The title and summary are<br>
> usually<br>
>    in the form of a :ref:`commit message <commit messages>`.<br>
> -* Add reviewers (see below for advice) and subscribe mailing<br>
> -  lists that you want to be included in the review. If your patch is<br>
> -  for LLVM, add llvm-commits as a Subscriber; if your patch is for Clang,<br>
> -  add cfe-commits.<br>
> +* Add reviewers (see below for advice). (If you set the Repository field<br>
> +  correctly, llvm-commits or cfe-commits will be subscribed<br>
> automatically;<br>
> +  otherwise, you will have to manually subscribe them.)<br>
>  * Leave the Repository and Project fields blank.<br>
<br>
Set the Repository field?  Then leave it blank?<br>
If there are two web forms with a Repository field and it matters which<br>
one is filled in, I think that ought to be clarified.  If this last<br>
bullet just shouldn't mention the Repository, please fix that.<br>
Thanks,<br>
--paulr<br>
<br>
>  * Click *Save*.<br>
><br>
><br>
><br>
> _______________________________________________<br>
> llvm-commits mailing list<br>
> <a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a><br>
> <a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
</blockquote></div></blockquote></div>