[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