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

Junmo Park via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 26 19:40:27 PST 2016

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?



More information about the llvm-commits mailing list