<div dir="ltr">Thanks. I've got a couple of post-review things I fixed up (comment, initialized InsertPt to EntryBB.end()).<div><br></div><div>That other review is interesting. I'll give it a better look when I get a chance.</div><div><br></div><div>-eric</div></div><br><div class="gmail_quote"><div dir="ltr">On Thu, Nov 19, 2015 at 4:29 PM Dan Gohman <<a href="mailto:sunfish@mozilla.com">sunfish@mozilla.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div>Hi Eric,<br><br></div>Yeah, looks ok to me. Thanks for fixing this!<br><br>As the comment says, I'm hoping to remove that code soon anyway; <a href="http://reviews.llvm.org/D14750" target="_blank">http://reviews.llvm.org/D14750</a> is what I'm currently hoping to do instead.<br><br></div></div><div dir="ltr">Dan<br><br></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Nov 19, 2015 at 4:24 PM, Eric Christopher <span dir="ltr"><<a href="mailto:echristo@gmail.com" target="_blank">echristo@gmail.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 Dan, JF,<div><br></div><div>I was getting a weird memory iterator error in this loop (in ToT with Debug+Asserts), but while I was looking at it I realized we could simplify the logic a bit if we split the loop apart into two - the part looking for the new insert point and then the part to look for all of the argument instructions.</div><div><br></div><div>It also solves the problem I was seeing with an invalid EntryBB.begin() during the insert.</div><div><br></div><div>OK?</div><span><font color="#888888"><div><br></div><div>-eric</div><div><br></div><div><br></div></font></span></div>
</blockquote></div><br></div>
</blockquote></div>