[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
More information about the llvm-commits