[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