<div dir="ltr">My theory is that in this specific instance (<a href="http://reviews.llvm.org/D4257">http://reviews.llvm.org/D4257</a>), the original sender is not a subscriber of llvm-commits.  Therefore the review request got held in moderation.  You can see in the archive that it (eventually) made it through here:<div>
<a href="http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20140623/223335.html">http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20140623/223335.html</a><br></div><div><br>I don't know where the initial email went.</div>
<div><br></div><div>I agree, it's really annoying how unreliable email has gotten lately.</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Jun 25, 2014 at 11:13 AM, Reid Kleckner <span dir="ltr"><<a href="mailto:rnk@google.com" target="_blank">rnk@google.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><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"><div class="">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>
</div><div><div class="h5"><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><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></div></div><br></div>
</blockquote></div><br></div>