[PATCH] D13259: LLE 6/6: Add LoopLoadElimination pass

hfinkel@anl.gov via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 2 10:37:03 PST 2015


hfinkel added a comment.

LGTM too, except for:

I don't think we check anywhere to disqualify volatile/atomic loads/stores, and we need to do that (especially for volatiles, since you're changing the number of accesses, and I'd need to think more-carefully about atomics).


http://reviews.llvm.org/D13259





More information about the llvm-commits mailing list