[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