<div dir="ltr">Ping. Matt added a test that fails with the implicit preconditions of the pass (i.e. that gathered and scattered both start as empty - this is now an assertion in the code as well). It also fixes the issue with returning false when changes have indeed been made to the underlying code.<div><br></div><div>Thanks,</div><div>Steve</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Jun 15, 2015 at 1:53 PM, Matt Wala <span dir="ltr"><<a href="mailto:wala@google.com" target="_blank">wala@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi Chandler:<div><br></div><div><span class="">I apologize, I did not mean to break any rules.<br></span><div class="gmail_extra"><br><div class="gmail_quote"><span class="">On Fri, Jun 12, 2015 at 5:50 PM, Chandler Carruth <span dir="ltr"><<a href="mailto:chandlerc@google.com" target="_blank">chandlerc@google.com</a>></span> wrote:<br></span><span class=""><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr">Matt, there are a bunch of problems here.<br><br>First, Steve Hines accepted this revision in Phabricator, but didn't actually send any LGTM or other comment to the mailing list, so it didn't look like *any* review happened.<div><br></div><div>Second, Steve, I'm not sure how familiar with this code you are as I've never seen you really contribute to it, so it might be better to let others review the patch.</div><div><br></div><div>Third, making the point that sufficient review had not happened, this is a bugfix patch which does not contain a test case. I feel confident that a dangling pointer style bug could be exercised with a good test case and ASan. We have ASan bots, so having the test case in the tree is really useful.</div><div><br></div><div>Matt, please update the review thread when you have added such a test case.</div></div></blockquote><div><br></div></span><span class=""><div>I've opened a new review with a test case attached.</div><div><br></div><div>Here's the link to the new review: <a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__reviews.llvm.org_D10459&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=mQ4LZ2PUj9hpadE3cDHZnIdEwhEBrbAstXeMaFoB9tg&m=Tv1U250hSfPBgwWqFA2MaisotfFOWTMJeZ7j6rHGHVU&s=A50ZEB2MlBI7--kMpMuxyF8aLVMuNmUEjFq9dk6m-fA&e=" target="_blank">http://reviews.llvm.org/D10459</a></div><div><br></div><div>Matt</div></span></div></div></div></div>
<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" rel="noreferrer" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
<br></blockquote></div><br></div>