[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