<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Jul 7, 2014 at 2:33 PM, Greg Clayton <span dir="ltr"><<a href="mailto:gclayton@apple.com" target="_blank" class="cremed">gclayton@apple.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Looks fine to me if the test suite is happy with the changes.<br></blockquote><div><br></div><div>No failures on Linux other than failures I've also seen on trunk (TestConvenienceVariables.py is consistently failing on Linux for me).</div>
<div><br></div><div>Willing to land it for me?  This is my first patch, so unless I'm told differently (:-}) I'm going with the LLVM process of submitting patches for a while rather than asking for commit access up front.  (I do intend to make multiple contributions to lldb over the coming weeks/months.)</div>
<div><br></div><div>Thanks!</div><div><br></div><div>-- Randy</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Greg<br>
<div><div class="h5"><br>
> On Jul 3, 2014, at 11:42 AM, Randy Smith <<a href="mailto:rdsmith@chromium.org" class="cremed">rdsmith@chromium.org</a>> wrote:<br>
><br>
><br>
> Based on r212225.  I've sketched out my rationale for this change in  <a href="http://llvm.org/bugs/show_bug.cgi?id=20149" target="_blank" class="cremed">http://llvm.org/bugs/show_bug.cgi?id=20149</a>.  It's fairly simple; is someone willing to take a look?<br>

><br>
> Note that I don't (yet) have commit access, so if this patch is approved someone will need to land it for me.  (I found the LLVM and LLDB pages not completely consistent around the process for gaining commit access, so I'm going with the LLVM "contribute several patches and then ask".)<br>

><br>
> I ran the "check-lldb" test suite on Linux, and found no difference in results between r212225 and r212225+my patch.  (As a side note, what should I expect from the Linux test suite?  TestConvenienceVariables.py is failing for me at top of tree.)<br>

><br>
> -- Randy<br>
><br>
</div></div>> <sourcedisp.diff>_______________________________________________<br>
> lldb-commits mailing list<br>
> <a href="mailto:lldb-commits@cs.uiuc.edu" class="cremed">lldb-commits@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits" target="_blank" class="cremed">http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits</a><br>
<br>
</blockquote></div><br></div></div>