<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Oct 3, 2014 at 10:28 AM, 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">Yes, the email address I use for phabricator is the same as this one.<br>
The email is also shown in the llvm-commits archive at:<br>
<a href="http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20140908/235087.html" target="_blank">http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20140908/235087.html</a></blockquote><div><br></div><div>So it is. Yeah, looks like it hit the mailing list just fine. Though your ping (if any) hasn't bumped that review thread - might be best to try that again (either by replying-all to the original review thread, or posting a 'ping' comment via Phabricator).</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
.  I'm not officially subscribed to here, but I guess I've emailed<br>
enough times, and/or because this is the email address registered with<br>
my llvm account, that I'm not getting bounces.<br>
<span class="HOEnZb"><font color="#888888"><br>
- Justin<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
On Fri, Oct 3, 2014 at 10:04 AM, David Blaikie <<a href="mailto:dblaikie@gmail.com">dblaikie@gmail.com</a>> wrote:<br>
> I don't see any other mail from you hitting the mailing list. Is the email<br>
> address you used with Phabricator the same one you're using in this thread?<br>
> (and/or if it's a different address, is that address subscribed to the<br>
> llvm-commits mailing list)<br>
><br>
> Without that subscription, the emails sent by phab will bounce off the<br>
> mailing list since they impersonate your address (& thus need taht address<br>
> to be subscribed to the mailing list).<br>
><br>
> - David<br>
><br>
> On Fri, Oct 3, 2014 at 8:33 AM, Justin Hibbits <<a href="mailto:jrh29@alumni.cwru.edu">jrh29@alumni.cwru.edu</a>><br>
> wrote:<br>
>><br>
>> llvm-commits is on the subscribers list.  I tried putting it on the<br>
>> reviewers list initially, but arc complained but allowed subscribers<br>
>> (I see it in the llvm-commits archive, too). I'll ping the thread<br>
>> directly, though.<br>
>><br>
>> - Justin<br>
>><br>
>> On Thu, 2 Oct 2014 23:57:38 -0700<br>
>> David Blaikie <<a href="mailto:dblaikie@gmail.com">dblaikie@gmail.com</a>> wrote:<br>
>><br>
>> > Doesn't look like the llvm-commits mailing list was ever added to the<br>
>> > reviewers.<br>
>> ><br>
>> > There's a particular quirk/problem with Phabricator reviews in<br>
>> > particular if you don't create them with the mailing list added to<br>
>> > the review on creation, then Phab never sends the proper introductory<br>
>> > mail if you add the mailing list later (the introductory mail is only<br>
>> > sent to those addresses the code review had at its creation).<br>
>> > Generally it's best to cancel a review that ends up in this state<br>
>> > (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<br>
>> > every week or so if it's not received attention. Some people are<br>
>> > busy, etc, this is an easy way to remind them that there's something<br>
>> > to do.<br>
>> ><br>
>> > So, my advice would be: cancel that review, create a new one and be<br>
>> > sure to put all the reviewers and especially the mailing list<br>
>> > (llvm-commits) on the review before you finish creating it. Make sure<br>
>> > that initial mail is sent to (and arrives on) the llvm-commits<br>
>> > mailing list. If there's no respons in a week, post a simple<br>
>> > 'ping' (either via Phab or just reply-all to the original Phab<br>
>> > mailing).<br>
>> ><br>
>> > On Thu, Oct 2, 2014 at 10:30 PM, Justin Hibbits<br>
>> > <<a href="mailto:jrh29@alumni.cwru.edu">jrh29@alumni.cwru.edu</a>> wrote:<br>
>> ><br>
>> > > 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<br>
>> > > that my other patches can go in: D5399 and D5400 (PowerPC -fpic<br>
>> > > support, and clang changes to support this, respectively).<br>
>> > ><br>
>> > > Thanks,<br>
>> > > Justin<br>
><br>
><br>
</div></div></blockquote></div><br></div></div>