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

[Junmo Park via llvm-commits]

flyingforyou marked an inline comment as done.
flyingforyou added a comment.

> Junmo, if my patch goes in, do you still need the check STn->isIndexed()? I hope to have mine merged soon.


Ana, I want to hear about opinion from Matt.
I don't want to disturb your effort in http://reviews.llvm.org/D15348. But the first test-case was made by Matt. And this commit's first reviewer is also Matt.

So, it's belong to him. Matt, could you give me a opinion about this change, please?

Junmo.


http://reviews.llvm.org/D16463