<div dir="ltr">Doesn't look like the llvm-commits mailing list was ever added to the reviewers.<br><br>There's a particular quirk/problem with Phabricator reviews in particular if you don't create them with the mailing list added to the review on creation, then Phab never sends the proper introductory mail if you add the mailing list later (the introductory mail is only sent to those addresses the code review had at its creation). Generally it's best to cancel a review that ends up in this state (since you can't get Phab to produce that email) and start a new one.<br><br>Beyond that, the usual recommendation is to 'ping' a review thread every week or so if it's not received attention. Some people are busy, etc, this is an easy way to remind them that there's something to do.<br><br>So, my advice would be: cancel that review, create a new one and be sure to put all the reviewers and especially the mailing list (llvm-commits) on the review before you finish creating it. Make sure that initial mail is sent to (and arrives on) the llvm-commits mailing list. If there's no respons in a week, post a simple 'ping' (either via Phab or just reply-all to the original Phab mailing).</div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Oct 2, 2014 at 10:30 PM, Justin Hibbits <span dir="ltr"><<a href="mailto:jrh29@alumni.cwru.edu" target="_blank">jrh29@alumni.cwru.edu</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Can somebody please review <a href="http://reviews.llvm.org/D5332" target="_blank">http://reviews.llvm.org/D5332</a>?  It's been<br>
sitting idle for nearly 3 weeks now, and I want to get it in so that my<br>
other patches can go in: D5399 and D5400 (PowerPC -fpic support, and<br>
clang changes to support this, respectively).<br>
<br>
Thanks,<br>
Justin<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
</blockquote></div><br></div>