[llvm] MemorySSA patch for 3.9

Daniel Berlin via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 20 17:45:18 PDT 2016


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)
It  should not be there for real, as it is definitely possible to have
cases where it is valid to visit > 1000 times.


On Wed, Jul 20, 2016 at 5:05 PM, George Burgess IV via llvm-commits <
llvm-commits at lists.llvm.org> wrote:

> Hi Hans,
>
> Can we please apply the attached patch to the 3.9 branch? It fixes PR28632,
> which is a release blocker.
>
> It doesn't apply to trunk, because we entirely replaced the piece of code
> that this patch "fixes" in r275940.
>
> (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. ;) )
>
> Thank you!
> George
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160720/152db872/attachment.html>


More information about the llvm-commits mailing list