<div dir="ltr"><div class="gmail_extra"><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.</blockquote>

<div><br></div><div>Phabricator has certainly had its share of technical difficulties lately. Just last week it suppressed all email to llvm-commits for many hours. These problems should be solved. That said, talking of "private reviews" and "back-channels" doesn't strike me as constructive. </div>

<div><br></div><div>Eli</div><div><br></div></div></div></div>