<div dir="ltr">Hi Chandler:<div><br></div><div>I apologize, I did not mean to break any rules.<br><div class="gmail_extra"><br><div class="gmail_quote">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><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><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=eK4ORRDb_othkP0-egROJhrwIm0LV6e1rr6Hvewuz9M&s=5tsojuA-2Bo8_bjRadJdtm7jn1NDfm_d1ZdpiU_ownU&e=">http://reviews.llvm.org/D10459</a></div><div><br></div><div>Matt</div></div></div></div></div>