[PATCH] D18909: Cleanup Store Merging in UseAA case

James Y Knight via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 11 06:46:57 PDT 2016


jyknight added a comment.

> Not one that highlights the difference. This is a factoring of some of the factorable cleanup of http://reviews.llvm.org/D14834. http://reviews.llvm.org/D14834 will remove the old path for store merging.


You ought to be able to detect that the chain is being set correctly now and wasn't before -- at the very least by looking at the nodes being produced.

Does this change by itself fix the test case in http://reviews.llvm.org/D17836?


http://reviews.llvm.org/D18909





More information about the llvm-commits mailing list