[PATCH] D18336: Prevent construction of cycle in DAG store merge

James Y Knight via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 25 14:02:45 PDT 2016


jyknight accepted this revision.
jyknight added a comment.
This revision is now accepted and ready to land.

Thanks, lgtm.

Note for readers: we spent some time going over this, and noted that 1) The original test case was using UseAA mode (which we'd not noticed before) 2) The bug doesn't actually exist in !UseAA, because of the different way it traverses the chain.

So this final version reflects that.


http://reviews.llvm.org/D18336





More information about the llvm-commits mailing list