[PATCH] D16463: [DAGCombiner] Don't add volatile or indexed stores to ChainedStores

Ana Pazos via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 25 21:01:33 PST 2016


apazos added a comment.

The fix I have is to allow forming store chains that can contain stores with pre/post increment. So the check for STn->isUnindexed can be removed from Junmo's patch since his is a more restrictive solution. Let's see if Junmo can create a test for the Volatile case only.


http://reviews.llvm.org/D16463





More information about the llvm-commits mailing list