[PATCH] D16463: [DAGCombiner] Don't add volatile or indexed stores to ChainedStores
Junmo Park via llvm-commits
llvm-commits at lists.llvm.org
Mon Jan 25 17:33:42 PST 2016
flyingforyou added a comment.
Hi Ana.
I am amazed that your first approch is almost same with my first patch.
But the last patch is extending `replaceStoreChain`. (This is not what I am doing in this patch.)
What is the proper decision about this patch? (Transfer the test case, Abandon?)
Let me know, please.
Junmo.
http://reviews.llvm.org/D16463
More information about the llvm-commits
mailing list