<div dir="ltr">Yeah, it was actually an assert i was using to make sure the code wasn't doing something broken on "normal" code while debugging the walker (it would occasionally infinite loop or do silly things when i first started writing it, and this was the easiest way to bugpoint things :P)<div><div>It  should not be there for real, as it is definitely possible to have cases where it is valid to visit > 1000 times.<br></div></div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Jul 20, 2016 at 5:05 PM, George Burgess IV via llvm-commits <span dir="ltr"><<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</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 Hans,<div><br></div><div>Can we please apply the attached patch to the 3.9 branch? It fixes <span style="white-space:pre-wrap;color:rgb(0,0,0)">PR28632, which is a release blocker.</span></div><div><br></div><div>It doesn't apply to trunk, because we entirely replaced the piece of code that this patch "fixes" in <span style="color:rgb(0,0,0);white-space:pre-wrap">r275940. </span></div><div><font color="#000000"><span style="white-space:pre-wrap"><br></span></font></div><div><font color="#000000"><span style="white-space:pre-wrap">(I realize that fixing an assertion failure by removing the assertion looks kind of bad, but said assertion was really more of a warning that we've hit a pathological-ish case than an error. ;) )</span></font></div><div><span style="color:rgb(0,0,0);white-space:pre-wrap"><br></span></div><div><span style="color:rgb(0,0,0);white-space:pre-wrap">Thank you!</span></div><span class="HOEnZb"><font color="#888888"><div><span style="color:rgb(0,0,0);white-space:pre-wrap">George</span></div></font></span></div>
<br>_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
<br></blockquote></div><br></div>