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

[Yin Ma via llvm-commits]

yinma added a comment.

Hi Nirav, I think the problem need to be solved should be the same with and without AA. It is testing the node predecessors. For your new upload, you use different check with UseAA.  For my algorithm, it only tests the latest node in the store array because it is the one that links with the successors. Do you have another test case that shows failure with UseAA? What could be the different with UseAA?


http://reviews.llvm.org/D18336