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

Yin Ma via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 24 13:42:56 PDT 2016

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?


More information about the llvm-commits mailing list