<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Aug 29, 2014 at 12:51 AM, Jason Molenda <span dir="ltr"><<a href="mailto:jason@molenda.com" target="_blank">jason@molenda.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hey Paul,<br>
<div class=""><br>
> <a href="http://reviews.llvm.org/D4579" target="_blank">http://reviews.llvm.org/D4579</a> - accepted, not commited<br>
<br>
</div>I assume this on Todd.<br></blockquote><div><br></div><div>I'll be looking into this one.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class=""><br>
> <a href="http://reviews.llvm.org/D4580" target="_blank">http://reviews.llvm.org/D4580</a> - waiting on Apple review<br>
<br>
</div>I replied separately but this looks good to me.<br>
<div class=""><br></div></blockquote><div><br></div><div>I'm just going to adjust the header includes and then will check in.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="">
> <a href="http://reviews.llvm.org/D4430" target="_blank">http://reviews.llvm.org/D4430</a> - waiting for review<br>
<br>
</div>Greg said on July 10 that this looks fine.  His reply didn't end up in the reviews history for some reason.  Do you think the error message "index %u is out of range since there are no targets listed" could be improved?  Maybe just "There are no active targets".  Or " ... out of range since there are no active targets".  Whatever, that's a minor nit, the patched version is clearly better.<br>

<div class=""><br></div></blockquote><div><br></div><div>Greg might need to make a <a href="http://reviews.llvm.org">reviews.llvm.org</a> phabricator account.  It might not mirror communication from non-members.  (If nothing else, there are definitely times where I want to add one or more of you guys as reviewers, generally Greg, but haven't been able to - in which case the mail just comes through the lldb-commits mirroring).</div>
<div><br></div><div>I'll wait to hear Paul's feedback before doing anything else with this one.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="">
> <a href="http://reviews.llvm.org/D5089" target="_blank">http://reviews.llvm.org/D5089</a> - newly added<br>
<br>
</div>I looked it over quickly & replied.  I had a couple of questions about RegisterContextPOSIXProcessMonitor_arm64::ReadRegister but otherwise it looked fine to me.<br>
<div class="HOEnZb"><div class="h5"><br></div></div></blockquote><div><br></div><div>I'll wait for this one to settle before touching.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="HOEnZb"><div class="h5">
<br>
<br>
On Aug 28, 2014, at 11:45 PM, Paul Osmialowski <<a href="mailto:pawelo@king.net.pl">pawelo@king.net.pl</a>> wrote:<br>
<br>
> Hi Todd,<br>
><br>
> I'm sorry for late response, I'm doing things on the run for last three<br>
> days, hopefully things will settle down soon. As for my patches I can see<br>
> that most of them are already commited. The ones that are waiting I listed<br>
> below:<br>
><br>
> <a href="http://reviews.llvm.org/D4579" target="_blank">http://reviews.llvm.org/D4579</a> - accepted, not commited<br>
> <a href="http://reviews.llvm.org/D4580" target="_blank">http://reviews.llvm.org/D4580</a> - waiting on Apple review<br>
> <a href="http://reviews.llvm.org/D4430" target="_blank">http://reviews.llvm.org/D4430</a> - waiting for review<br>
> <a href="http://reviews.llvm.org/D5089" target="_blank">http://reviews.llvm.org/D5089</a> - newly added<br>
><br>
> Cheers,<br>
> Paul<br>
><br>
> <a href="http://reviews.llvm.org/D4381" target="_blank">http://reviews.llvm.org/D4381</a><br>
><br>
><br>
><br>
> _______________________________________________<br>
> lldb-commits mailing list<br>
> <a href="mailto:lldb-commits@cs.uiuc.edu">lldb-commits@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits</a><br>
<br>
<br>
_______________________________________________<br>
lldb-commits mailing list<br>
<a href="mailto:lldb-commits@cs.uiuc.edu">lldb-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits</a><br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div dir="ltr"><table cellspacing="0" cellpadding="0" style="color:rgb(136,136,136);font-family:'Times New Roman'"><tbody><tr style="color:rgb(85,85,85);font-family:sans-serif;font-size:small">
<td nowrap style="border-top-style:solid;border-top-color:rgb(213,15,37);border-top-width:2px">Todd Fiala |</td><td nowrap style="border-top-style:solid;border-top-color:rgb(51,105,232);border-top-width:2px"> Software Engineer |</td>
<td nowrap style="border-top-style:solid;border-top-color:rgb(0,153,57);border-top-width:2px"> <a href="mailto:tfiala@google.com" style="color:rgb(17,85,204)" target="_blank"><span style="background-color:rgb(255,255,204);color:rgb(34,34,34);background-repeat:initial initial">tfiala@google.com</span></a> |</td>
<td nowrap style="border-top-style:solid;border-top-color:rgb(238,178,17);border-top-width:2px"><font color="#1155cc"> <a>650-943-3180</a></font></td></tr></tbody></table><br></div>
</div></div>