<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">http://reviews.llvm.org/D14750</a> is what I'm currently hoping to do instead.<br><br></div>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 class="HOEnZb"><font color="#888888"><div><br></div><div>-eric</div><div><br></div><div><br></div></font></span></div>
</blockquote></div><br></div>