<div><div dir="auto">The people who are going to review this patch would prefer the web interface. If you look at this mailing list history you will see that the vast majority of the patches go through phabricator. The documentation on the webpage is a little out of date with how strongly we prefer to use phabricator these days.</div></div><div><br><div class="gmail_quote"><div dir="ltr">On Tue, Dec 11, 2018 at 10:51 AM Stefan Kanthak via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">"Roman Lebedev" <<a href="mailto:lebedev.ri@gmail.com" target="_blank">lebedev.ri@gmail.com</a>> wrote:<br>
<br>
> On Tue, Dec 11, 2018 at 8:58 PM Stefan Kanthak via llvm-commits<br>
> <<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>> wrote:<br>
>><br>
>> LLVM/clang inlines "long" shifts, so __udivmoddi4.c doesn't need<br>
>> several if...then...else...if...then...else clauses and can be<br>
>> simplified, yielding shorter and faster code.<br>
> Can you please submit these patches via the phabricator please, every<br>
> review is done via it.<br>
> <a href="https://llvm.org/docs/Phabricator.html" rel="noreferrer" target="_blank">https://llvm.org/docs/Phabricator.html</a><br>
<br>
Citing that page:<br>
<br>
| If you prefer to use a web user interface for code reviews, ...<br>
<br>
I don't prefer a web interface, I prefer plain old SMTP [*]!<br>
<br>
| While Phabricator is a useful tool for some, the relevant -commits<br>
| mailing list is the system of record for all LLVM code review.<br>
<br>
Phabricator is no useful tool for me.<br>
<br>
[*] <<a href="https://llvm.org/docs/DeveloperPolicy.html" rel="noreferrer" target="_blank">https://llvm.org/docs/DeveloperPolicy.html</a>><br>
| Once your patch is ready, submit it by emailing it to the appropriate<br>
| project's commit mailing list (or commit it directly if applicable).<br>
| Alternatively, some patches get sent to the project's development list<br>
| or component of the LLVM bug tracker, but the commit list is the<br>
                                            ~~~~~~~~~~~~~~~~~~~~~~<br>
| primary place for reviews and should generally be preferred.<br>
  ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~<br>
<br>
According to LLVM's own documentation, submission of patches via<br>
*-commits is the preferred method.<br>
<br>
Stefan<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></div>-- <br><div dir="ltr" class="gmail_signature" data-smartmail="gmail_signature">~Craig</div>