<div dir="ltr">Can you provide some data to support this by comparing commits with phab URLs in them with the llvm-commits archive?<div><br></div><div>It may be that llvm-commits is properly forwarding the review mail, but it's getting caught in people's spam filters.  I've personally had problems with this.</div>
</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Jun 25, 2014 at 10:44 AM, Alp Toker <span dir="ltr"><<a href="mailto:alp@nuanti.com" target="_blank">alp@nuanti.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
For whatever reason, patches posted to the Phabricator website still aren't being sent to the mailing list, making it difficult for us to review them.<br>
<br>
I've raised this issue a couple of times in the last few weeks.<br>
<br>
In practice this has a detrimental effect to the development workflow because it means that code is being seen only by a small group of individuals who have web accounts. The code isn't hitting llvm-commits or cfe-commits where the majority of code maintainers use the mailing lists for review.<br>

<br>
At this point I think Phabricator should be disabled and patches should be send to the mailing lists *until* the technical issue is confirmed resolved.<br>
<br>
It's really uncool that code is entering ToT through this back-channel -- I appreciate that it might not be intentional, but every single patch that gets committed this way is a real problem for the project.<span class="HOEnZb"><font color="#888888"><br>

<br>
Alp.<br>
<br>
-- <br>
<a href="http://www.nuanti.com" target="_blank">http://www.nuanti.com</a><br>
the browser experts<br>
<br>
______________________________<u></u>_________________<br>
LLVM Developers mailing list<br>
<a href="mailto:LLVMdev@cs.uiuc.edu" target="_blank">LLVMdev@cs.uiuc.edu</a>         <a href="http://llvm.cs.uiuc.edu" target="_blank">http://llvm.cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev" target="_blank">http://lists.cs.uiuc.edu/<u></u>mailman/listinfo/llvmdev</a><br>
</font></span></blockquote></div><br></div>