[PATCH] D13379: Add an utility to lift aggregate loads in SROA.

Chandler Carruth via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 7 18:40:12 PDT 2015


chandlerc added a comment.

As I mentioned on the original thread, SROA already has logic to remove FCA loads and stores. We shouldn't *just* add more code to SROA, we should re-use (and possibly extend) the existing code.

Does this make sense?

I'm still not sure that we should *only* do this inside of SROA, but maybe that's enough...


http://reviews.llvm.org/D13379





More information about the llvm-commits mailing list