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

Matt Arsenault via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 25 17:41:41 PST 2016


arsenm added a comment.

In http://reviews.llvm.org/D16463#335628, @apazos wrote:

> Yes, it looks like the issue in http://reviews.llvm.org/D15348. Matt can you try your reduced test with that on-going review?


I don't think that patch will fix this bug. It will fix the assertion, but I think this problem uncovers one that will also effect volatiles. You can probably produce a testcase derived from this one which will break volatile handling


http://reviews.llvm.org/D16463





More information about the llvm-commits mailing list