[PATCH] D31068: [SDAG] Expand MergedConsecutiveStores to better handle Giving up in Chain Analysis

Nirav Dave via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 11 05:41:57 PDT 2017


niravd added a comment.

I'm a little concerned about the selection candidates having a chain loop dependence after mething. Can you get a llvm testcase I can look at myself. Alternatively I can probably cobble one myself if I can get a snapshot of the DAG right before the relevant merge

In https://reviews.llvm.org/D31068#723192, @aditya_nandakumar wrote:

> Nirav - I posted the reason for the regression and a quick fix for the issue I saw in https://reviews.llvm.org/D31914.





https://reviews.llvm.org/D31068





More information about the llvm-commits mailing list